/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to plugins.d/mandos-client.c

  • Committer: Teddy Hogeborn
  • Date: 2014-07-15 19:35:03 UTC
  • Revision ID: teddy@recompile.se-20140715193503-e18hls7m4rv4l6iq
mandos-client: Fix mem free bug.

* plugins.d/mandos-client.c (add_server): Hide warning.
  (main): Free server->ip too, not just the server struct.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008-2013 Teddy Hogeborn
13
 
 * Copyright © 2008-2013 Björn Påhlsson
 
12
 * Copyright © 2008-2014 Teddy Hogeborn
 
13
 * Copyright © 2008-2014 Björn Påhlsson
14
14
 * 
15
15
 * This program is free software: you can redistribute it and/or
16
16
 * modify it under the terms of the GNU General Public License as
32
32
/* Needed by GPGME, specifically gpgme_data_seek() */
33
33
#ifndef _LARGEFILE_SOURCE
34
34
#define _LARGEFILE_SOURCE
35
 
#endif
 
35
#endif  /* not _LARGEFILE_SOURCE */
36
36
#ifndef _FILE_OFFSET_BITS
37
37
#define _FILE_OFFSET_BITS 64
38
 
#endif
 
38
#endif  /* not _FILE_OFFSET_BITS */
39
39
 
40
40
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), asprintf() */
41
41
 
42
42
#include <stdio.h>              /* fprintf(), stderr, fwrite(),
43
 
                                   stdout, ferror(), remove() */
 
43
                                   stdout, ferror() */
44
44
#include <stdint.h>             /* uint16_t, uint32_t, intptr_t */
45
45
#include <stddef.h>             /* NULL, size_t, ssize_t */
46
46
#include <stdlib.h>             /* free(), EXIT_SUCCESS, srand(),
57
57
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
58
58
                                   inet_pton(), connect(),
59
59
                                   getnameinfo() */
60
 
#include <fcntl.h>              /* open() */
 
60
#include <fcntl.h>              /* open(), unlinkat() */
61
61
#include <dirent.h>             /* opendir(), struct dirent, readdir()
62
62
                                 */
63
63
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
73
73
                                */
74
74
#include <unistd.h>             /* close(), SEEK_SET, off_t, write(),
75
75
                                   getuid(), getgid(), seteuid(),
76
 
                                   setgid(), pause(), _exit() */
 
76
                                   setgid(), pause(), _exit(),
 
77
                                   unlinkat() */
77
78
#include <arpa/inet.h>          /* inet_pton(), htons() */
78
79
#include <iso646.h>             /* not, or, and */
79
80
#include <argp.h>               /* struct argp_option, error_t, struct
141
142
static const char sys_class_net[] = "/sys/class/net";
142
143
char *connect_to = NULL;
143
144
const char *hookdir = HOOKDIR;
 
145
int hookdir_fd = -1;
144
146
uid_t uid = 65534;
145
147
gid_t gid = 65534;
146
148
 
232
234
                          .af = af };
233
235
  if(new_server->ip == NULL){
234
236
    perror_plus("strdup");
 
237
    free(new_server);
235
238
    return false;
236
239
  }
237
240
  ret = clock_gettime(CLOCK_MONOTONIC, &(new_server->last_seen));
238
241
  if(ret == -1){
239
242
    perror_plus("clock_gettime");
 
243
#ifdef __GNUC__
 
244
#pragma GCC diagnostic push
 
245
#pragma GCC diagnostic ignored "-Wcast-qual"
 
246
#endif
 
247
    free((char *)(new_server->ip));
 
248
#ifdef __GNUC__
 
249
#pragma GCC diagnostic pop
 
250
#endif
 
251
    free(new_server);
240
252
    return false;
241
253
  }
242
254
  /* Special case of first server */
258
270
 * Initialize GPGME.
259
271
 */
260
272
__attribute__((nonnull, warn_unused_result))
261
 
static bool init_gpgme(const char *seckey, const char *pubkey,
262
 
                       const char *tempdir, mandos_context *mc){
 
273
static bool init_gpgme(const char * const seckey,
 
274
                       const char * const pubkey,
 
275
                       const char * const tempdir,
 
276
                       mandos_context *mc){
263
277
  gpgme_error_t rc;
264
278
  gpgme_engine_info_t engine_info;
265
279
  
266
280
  /*
267
281
   * Helper function to insert pub and seckey to the engine keyring.
268
282
   */
269
 
  bool import_key(const char *filename){
 
283
  bool import_key(const char * const filename){
270
284
    int ret;
271
285
    int fd;
272
286
    gpgme_data_t pgp_data;
1333
1347
  sret = strspn(direntry->d_name, "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
1334
1348
                "abcdefghijklmnopqrstuvwxyz"
1335
1349
                "0123456789"
1336
 
                "_-");
 
1350
                "_.-");
1337
1351
  if((direntry->d_name)[sret] != '\0'){
1338
1352
    /* Contains non-allowed characters */
1339
1353
    if(debug){
1343
1357
    return 0;
1344
1358
  }
1345
1359
  
1346
 
  char *fullname = NULL;
1347
 
  ret = asprintf(&fullname, "%s/%s", hookdir, direntry->d_name);
1348
 
  if(ret < 0){
1349
 
    perror_plus("asprintf");
1350
 
    return 0;
1351
 
  }
1352
 
  
1353
 
  ret = stat(fullname, &st);
 
1360
  ret = fstatat(hookdir_fd, direntry->d_name, &st, 0);
1354
1361
  if(ret == -1){
1355
1362
    if(debug){
1356
1363
      perror_plus("Could not stat hook");
1461
1468
  error_t ret_errno = 0;
1462
1469
  if(seteuid(0) == -1){
1463
1470
    ret_errno = errno;
1464
 
    perror_plus("seteuid");
1465
1471
  }
1466
1472
  errno = old_errno;
1467
1473
  return ret_errno;
1478
1484
  }
1479
1485
  if(setuid(0) == -1){
1480
1486
    ret_errno = errno;
1481
 
    perror_plus("seteuid");
1482
1487
  }
1483
1488
  errno = old_errno;
1484
1489
  return ret_errno;
1491
1496
  error_t ret_errno = 0;
1492
1497
  if(seteuid(uid) == -1){
1493
1498
    ret_errno = errno;
1494
 
    perror_plus("seteuid");
1495
1499
  }
1496
1500
  errno = old_errno;
1497
1501
  return ret_errno;
1504
1508
  error_t ret_errno = 0;
1505
1509
  if(setuid(uid) == -1){
1506
1510
    ret_errno = errno;
1507
 
    perror_plus("setuid");
1508
1511
  }
1509
1512
  errno = old_errno;
1510
1513
  return ret_errno;
1513
1516
__attribute__((nonnull))
1514
1517
void run_network_hooks(const char *mode, const char *interface,
1515
1518
                       const float delay){
1516
 
  struct dirent **direntries;
1517
 
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
1518
 
                         alphasort);
 
1519
  struct dirent **direntries = NULL;
 
1520
  if(hookdir_fd == -1){
 
1521
    hookdir_fd = open(hookdir, O_RDONLY);
 
1522
    if(hookdir_fd == -1){
 
1523
      if(errno == ENOENT){
 
1524
        if(debug){
 
1525
          fprintf_plus(stderr, "Network hook directory \"%s\" not"
 
1526
                       " found\n", hookdir);
 
1527
        }
 
1528
      } else {
 
1529
        perror_plus("open");
 
1530
      }
 
1531
      return;
 
1532
    }
 
1533
  }
 
1534
#ifdef __GLIBC__
 
1535
#if __GLIBC_PREREQ(2, 15)
 
1536
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
 
1537
                           runnable_hook, alphasort);
 
1538
#else  /* not __GLIBC_PREREQ(2, 15) */
 
1539
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1540
                         alphasort);
 
1541
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
1542
#else   /* not __GLIBC__ */
 
1543
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1544
                         alphasort);
 
1545
#endif  /* not __GLIBC__ */
1519
1546
  if(numhooks == -1){
1520
 
    if(errno == ENOENT){
1521
 
      if(debug){
1522
 
        fprintf_plus(stderr, "Network hook directory \"%s\" not"
1523
 
                     " found\n", hookdir);
1524
 
      }
1525
 
    } else {
1526
 
      perror_plus("scandir");
 
1547
    perror_plus("scandir");
 
1548
    return;
 
1549
  }
 
1550
  struct dirent *direntry;
 
1551
  int ret;
 
1552
  int devnull = open("/dev/null", O_RDONLY);
 
1553
  for(int i = 0; i < numhooks; i++){
 
1554
    direntry = direntries[i];
 
1555
    if(debug){
 
1556
      fprintf_plus(stderr, "Running network hook \"%s\"\n",
 
1557
                   direntry->d_name);
1527
1558
    }
1528
 
  } else {
1529
 
    struct dirent *direntry;
1530
 
    int ret;
1531
 
    int devnull = open("/dev/null", O_RDONLY);
1532
 
    for(int i = 0; i < numhooks; i++){
1533
 
      direntry = direntries[i];
1534
 
      char *fullname = NULL;
1535
 
      ret = asprintf(&fullname, "%s/%s", hookdir, direntry->d_name);
1536
 
      if(ret < 0){
 
1559
    pid_t hook_pid = fork();
 
1560
    if(hook_pid == 0){
 
1561
      /* Child */
 
1562
      /* Raise privileges */
 
1563
      errno = raise_privileges_permanently();
 
1564
      if(errno != 0){
 
1565
        perror_plus("Failed to raise privileges");
 
1566
        _exit(EX_NOPERM);
 
1567
      }
 
1568
      /* Set group */
 
1569
      errno = 0;
 
1570
      ret = setgid(0);
 
1571
      if(ret == -1){
 
1572
        perror_plus("setgid");
 
1573
        _exit(EX_NOPERM);
 
1574
      }
 
1575
      /* Reset supplementary groups */
 
1576
      errno = 0;
 
1577
      ret = setgroups(0, NULL);
 
1578
      if(ret == -1){
 
1579
        perror_plus("setgroups");
 
1580
        _exit(EX_NOPERM);
 
1581
      }
 
1582
      ret = dup2(devnull, STDIN_FILENO);
 
1583
      if(ret == -1){
 
1584
        perror_plus("dup2(devnull, STDIN_FILENO)");
 
1585
        _exit(EX_OSERR);
 
1586
      }
 
1587
      ret = close(devnull);
 
1588
      if(ret == -1){
 
1589
        perror_plus("close");
 
1590
        _exit(EX_OSERR);
 
1591
      }
 
1592
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
 
1593
      if(ret == -1){
 
1594
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
 
1595
        _exit(EX_OSERR);
 
1596
      }
 
1597
      ret = setenv("MANDOSNETHOOKDIR", hookdir, 1);
 
1598
      if(ret == -1){
 
1599
        perror_plus("setenv");
 
1600
        _exit(EX_OSERR);
 
1601
      }
 
1602
      ret = setenv("DEVICE", interface, 1);
 
1603
      if(ret == -1){
 
1604
        perror_plus("setenv");
 
1605
        _exit(EX_OSERR);
 
1606
      }
 
1607
      ret = setenv("VERBOSITY", debug ? "1" : "0", 1);
 
1608
      if(ret == -1){
 
1609
        perror_plus("setenv");
 
1610
        _exit(EX_OSERR);
 
1611
      }
 
1612
      ret = setenv("MODE", mode, 1);
 
1613
      if(ret == -1){
 
1614
        perror_plus("setenv");
 
1615
        _exit(EX_OSERR);
 
1616
      }
 
1617
      char *delaystring;
 
1618
      ret = asprintf(&delaystring, "%f", (double)delay);
 
1619
      if(ret == -1){
1537
1620
        perror_plus("asprintf");
1538
 
        continue;
1539
 
      }
1540
 
      if(debug){
1541
 
        fprintf_plus(stderr, "Running network hook \"%s\"\n",
1542
 
                     direntry->d_name);
1543
 
      }
1544
 
      pid_t hook_pid = fork();
1545
 
      if(hook_pid == 0){
1546
 
        /* Child */
1547
 
        /* Raise privileges */
1548
 
        if(raise_privileges_permanently() != 0){
1549
 
          perror_plus("Failed to raise privileges");
1550
 
          _exit(EX_NOPERM);
1551
 
        }
1552
 
        /* Set group */
1553
 
        errno = 0;
1554
 
        ret = setgid(0);
1555
 
        if(ret == -1){
1556
 
          perror_plus("setgid");
1557
 
          _exit(EX_NOPERM);
1558
 
        }
1559
 
        /* Reset supplementary groups */
1560
 
        errno = 0;
1561
 
        ret = setgroups(0, NULL);
1562
 
        if(ret == -1){
1563
 
          perror_plus("setgroups");
1564
 
          _exit(EX_NOPERM);
1565
 
        }
1566
 
        ret = dup2(devnull, STDIN_FILENO);
1567
 
        if(ret == -1){
1568
 
          perror_plus("dup2(devnull, STDIN_FILENO)");
1569
 
          _exit(EX_OSERR);
1570
 
        }
1571
 
        ret = close(devnull);
1572
 
        if(ret == -1){
1573
 
          perror_plus("close");
1574
 
          _exit(EX_OSERR);
1575
 
        }
1576
 
        ret = dup2(STDERR_FILENO, STDOUT_FILENO);
1577
 
        if(ret == -1){
1578
 
          perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
1579
 
          _exit(EX_OSERR);
1580
 
        }
1581
 
        ret = setenv("MANDOSNETHOOKDIR", hookdir, 1);
1582
 
        if(ret == -1){
1583
 
          perror_plus("setenv");
1584
 
          _exit(EX_OSERR);
1585
 
        }
1586
 
        ret = setenv("DEVICE", interface, 1);
1587
 
        if(ret == -1){
1588
 
          perror_plus("setenv");
1589
 
          _exit(EX_OSERR);
1590
 
        }
1591
 
        ret = setenv("VERBOSITY", debug ? "1" : "0", 1);
1592
 
        if(ret == -1){
1593
 
          perror_plus("setenv");
1594
 
          _exit(EX_OSERR);
1595
 
        }
1596
 
        ret = setenv("MODE", mode, 1);
1597
 
        if(ret == -1){
1598
 
          perror_plus("setenv");
1599
 
          _exit(EX_OSERR);
1600
 
        }
1601
 
        char *delaystring;
1602
 
        ret = asprintf(&delaystring, "%f", (double)delay);
1603
 
        if(ret == -1){
1604
 
          perror_plus("asprintf");
1605
 
          _exit(EX_OSERR);
1606
 
        }
1607
 
        ret = setenv("DELAY", delaystring, 1);
1608
 
        if(ret == -1){
1609
 
          free(delaystring);
1610
 
          perror_plus("setenv");
1611
 
          _exit(EX_OSERR);
1612
 
        }
 
1621
        _exit(EX_OSERR);
 
1622
      }
 
1623
      ret = setenv("DELAY", delaystring, 1);
 
1624
      if(ret == -1){
1613
1625
        free(delaystring);
1614
 
        if(connect_to != NULL){
1615
 
          ret = setenv("CONNECT", connect_to, 1);
1616
 
          if(ret == -1){
1617
 
            perror_plus("setenv");
1618
 
            _exit(EX_OSERR);
1619
 
          }
1620
 
        }
1621
 
        if(execl(fullname, direntry->d_name, mode, NULL) == -1){
1622
 
          perror_plus("execl");
1623
 
          _exit(EXIT_FAILURE);
1624
 
        }
 
1626
        perror_plus("setenv");
 
1627
        _exit(EX_OSERR);
 
1628
      }
 
1629
      free(delaystring);
 
1630
      if(connect_to != NULL){
 
1631
        ret = setenv("CONNECT", connect_to, 1);
 
1632
        if(ret == -1){
 
1633
          perror_plus("setenv");
 
1634
          _exit(EX_OSERR);
 
1635
        }
 
1636
      }
 
1637
      int hook_fd = openat(hookdir_fd, direntry->d_name, O_RDONLY);
 
1638
      if(hook_fd == -1){
 
1639
        perror_plus("openat");
 
1640
        _exit(EXIT_FAILURE);
 
1641
      }
 
1642
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
1643
        perror_plus("close");
 
1644
        _exit(EXIT_FAILURE);
 
1645
      }
 
1646
      if(fexecve(hook_fd, (char *const []){ direntry->d_name, NULL },
 
1647
                 environ) == -1){
 
1648
        perror_plus("fexecve");
 
1649
        _exit(EXIT_FAILURE);
 
1650
      }
 
1651
    } else {
 
1652
      int status;
 
1653
      if(TEMP_FAILURE_RETRY(waitpid(hook_pid, &status, 0)) == -1){
 
1654
        perror_plus("waitpid");
 
1655
        continue;
 
1656
      }
 
1657
      if(WIFEXITED(status)){
 
1658
        if(WEXITSTATUS(status) != 0){
 
1659
          fprintf_plus(stderr, "Warning: network hook \"%s\" exited"
 
1660
                       " with status %d\n", direntry->d_name,
 
1661
                       WEXITSTATUS(status));
 
1662
          continue;
 
1663
        }
 
1664
      } else if(WIFSIGNALED(status)){
 
1665
        fprintf_plus(stderr, "Warning: network hook \"%s\" died by"
 
1666
                     " signal %d\n", direntry->d_name,
 
1667
                     WTERMSIG(status));
 
1668
        continue;
1625
1669
      } else {
1626
 
        int status;
1627
 
        if(TEMP_FAILURE_RETRY(waitpid(hook_pid, &status, 0)) == -1){
1628
 
          perror_plus("waitpid");
1629
 
          free(fullname);
1630
 
          continue;
1631
 
        }
1632
 
        if(WIFEXITED(status)){
1633
 
          if(WEXITSTATUS(status) != 0){
1634
 
            fprintf_plus(stderr, "Warning: network hook \"%s\" exited"
1635
 
                         " with status %d\n", direntry->d_name,
1636
 
                         WEXITSTATUS(status));
1637
 
            free(fullname);
1638
 
            continue;
1639
 
          }
1640
 
        } else if(WIFSIGNALED(status)){
1641
 
          fprintf_plus(stderr, "Warning: network hook \"%s\" died by"
1642
 
                       " signal %d\n", direntry->d_name,
1643
 
                       WTERMSIG(status));
1644
 
          free(fullname);
1645
 
          continue;
1646
 
        } else {
1647
 
          fprintf_plus(stderr, "Warning: network hook \"%s\""
1648
 
                       " crashed\n", direntry->d_name);
1649
 
          free(fullname);
1650
 
          continue;
1651
 
        }
1652
 
      }
1653
 
      free(fullname);
1654
 
      if(debug){
1655
 
        fprintf_plus(stderr, "Network hook \"%s\" ran successfully\n",
1656
 
                     direntry->d_name);
1657
 
      }
1658
 
    }
1659
 
    close(devnull);
1660
 
  }
 
1670
        fprintf_plus(stderr, "Warning: network hook \"%s\""
 
1671
                     " crashed\n", direntry->d_name);
 
1672
        continue;
 
1673
      }
 
1674
    }
 
1675
    if(debug){
 
1676
      fprintf_plus(stderr, "Network hook \"%s\" ran successfully\n",
 
1677
                   direntry->d_name);
 
1678
    }
 
1679
  }
 
1680
  free(direntries);
 
1681
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
1682
    perror_plus("close");
 
1683
  } else {
 
1684
    hookdir_fd = -1;
 
1685
  }
 
1686
  close(devnull);
1661
1687
}
1662
1688
 
1663
1689
__attribute__((nonnull, warn_unused_result))
1713
1739
    
1714
1740
    /* Raise privileges */
1715
1741
    ret_errno = raise_privileges();
 
1742
    if(ret_errno != 0){
 
1743
      errno = ret_errno;
 
1744
      perror_plus("Failed to raise privileges");
 
1745
    }
 
1746
    
 
1747
#ifdef __linux__
 
1748
    int ret_linux;
1716
1749
    bool restore_loglevel = false;
1717
 
    if(ret_errno != 0){
1718
 
      perror_plus("Failed to raise privileges");
1719
 
    }
1720
 
#ifdef __linux__
1721
 
    int ret_linux;
1722
1750
    if(ret_errno == 0){
1723
1751
      /* Lower kernel loglevel to KERN_NOTICE to avoid KERN_INFO
1724
1752
         messages about the network interface to mess up the prompt */
1822
1850
    /* Raise privileges */
1823
1851
    ret_errno = raise_privileges();
1824
1852
    if(ret_errno != 0){
 
1853
      errno = ret_errno;
1825
1854
      perror_plus("Failed to raise privileges");
1826
1855
    }
 
1856
    
1827
1857
    int ret_setflags = ioctl(sd, SIOCSIFFLAGS, &network);
1828
1858
    ioctl_errno = errno;
1829
1859
    
1871
1901
  int exitcode = EXIT_SUCCESS;
1872
1902
  char *interfaces_to_take_down = NULL;
1873
1903
  size_t interfaces_to_take_down_size = 0;
1874
 
  char tempdir[] = "/tmp/mandosXXXXXX";
1875
 
  bool tempdir_created = false;
 
1904
  char run_tempdir[] = "/run/tmp/mandosXXXXXX";
 
1905
  char old_tempdir[] = "/tmp/mandosXXXXXX";
 
1906
  char *tempdir = NULL;
1876
1907
  AvahiIfIndex if_index = AVAHI_IF_UNSPEC;
1877
1908
  const char *seckey = PATHDIR "/" SECKEY;
1878
1909
  const char *pubkey = PATHDIR "/" PUBKEY;
2232
2263
  
2233
2264
  /* If no interfaces were specified, make a list */
2234
2265
  if(mc.interfaces == NULL){
2235
 
    struct dirent **direntries;
 
2266
    struct dirent **direntries = NULL;
2236
2267
    /* Look for any good interfaces */
2237
2268
    ret = scandir(sys_class_net, &direntries, good_interface,
2238
2269
                  alphasort);
2253
2284
      }
2254
2285
      free(direntries);
2255
2286
    } else {
2256
 
      free(direntries);
 
2287
      if(ret == 0){
 
2288
        free(direntries);
 
2289
      }
2257
2290
      fprintf_plus(stderr, "Could not find a network interface\n");
2258
2291
      exitcode = EXIT_FAILURE;
2259
2292
      goto end;
2327
2360
    goto end;
2328
2361
  }
2329
2362
  
2330
 
  if(mkdtemp(tempdir) == NULL){
 
2363
  /* Try /run/tmp before /tmp */
 
2364
  tempdir = mkdtemp(run_tempdir);
 
2365
  if(tempdir == NULL and errno == ENOENT){
 
2366
      if(debug){
 
2367
        fprintf_plus(stderr, "Tempdir %s did not work, trying %s\n",
 
2368
                     run_tempdir, old_tempdir);
 
2369
      }
 
2370
      tempdir = mkdtemp(old_tempdir);
 
2371
  }
 
2372
  if(tempdir == NULL){
2331
2373
    perror_plus("mkdtemp");
2332
2374
    goto end;
2333
2375
  }
2334
 
  tempdir_created = true;
2335
2376
  
2336
2377
  if(quit_now){
2337
2378
    goto end;
2473
2514
  if(debug){
2474
2515
    fprintf_plus(stderr, "Starting Avahi loop search\n");
2475
2516
  }
2476
 
 
 
2517
  
2477
2518
  ret = avahi_loop_with_timeout(simple_poll,
2478
2519
                                (int)(retry_interval * 1000), &mc);
2479
2520
  if(debug){
2515
2556
    mc.current_server->prev->next = NULL;
2516
2557
    while(mc.current_server != NULL){
2517
2558
      server *next = mc.current_server->next;
 
2559
#ifdef __GNUC__
 
2560
#pragma GCC diagnostic push
 
2561
#pragma GCC diagnostic ignored "-Wcast-qual"
 
2562
#endif
 
2563
      free((char *)(mc.current_server->ip));
 
2564
#ifdef __GNUC__
 
2565
#pragma GCC diagnostic pop
 
2566
#endif
2518
2567
      free(mc.current_server);
2519
2568
      mc.current_server = next;
2520
2569
    }
2524
2573
  {
2525
2574
    ret_errno = raise_privileges();
2526
2575
    if(ret_errno != 0){
 
2576
      errno = ret_errno;
2527
2577
      perror_plus("Failed to raise privileges");
2528
2578
    } else {
2529
2579
      
2549
2599
        }
2550
2600
      }
2551
2601
    }
 
2602
    
2552
2603
    ret_errno = lower_privileges_permanently();
2553
2604
    if(ret_errno != 0){
 
2605
      errno = ret_errno;
2554
2606
      perror_plus("Failed to lower privileges permanently");
2555
2607
    }
2556
2608
  }
2559
2611
  free(interfaces_hooks);
2560
2612
  
2561
2613
  /* Removes the GPGME temp directory and all files inside */
2562
 
  if(tempdir_created){
 
2614
  if(tempdir != NULL){
2563
2615
    struct dirent **direntries = NULL;
2564
 
    struct dirent *direntry = NULL;
2565
 
    int numentries = scandir(tempdir, &direntries, notdotentries,
2566
 
                             alphasort);
2567
 
    if(numentries > 0){
2568
 
      for(int i = 0; i < numentries; i++){
2569
 
        direntry = direntries[i];
2570
 
        char *fullname = NULL;
2571
 
        ret = asprintf(&fullname, "%s/%s", tempdir,
2572
 
                       direntry->d_name);
2573
 
        if(ret < 0){
2574
 
          perror_plus("asprintf");
2575
 
          continue;
2576
 
        }
2577
 
        ret = remove(fullname);
2578
 
        if(ret == -1){
2579
 
          fprintf_plus(stderr, "remove(\"%s\"): %s\n", fullname,
2580
 
                       strerror(errno));
2581
 
        }
2582
 
        free(fullname);
 
2616
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY |
 
2617
                                                  O_NOFOLLOW));
 
2618
    if(tempdir_fd == -1){
 
2619
      perror_plus("open");
 
2620
    } else {
 
2621
#ifdef __GLIBC__
 
2622
#if __GLIBC_PREREQ(2, 15)
 
2623
      int numentries = scandirat(tempdir_fd, ".", &direntries,
 
2624
                                 notdotentries, alphasort);
 
2625
#else  /* not __GLIBC_PREREQ(2, 15) */
 
2626
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
2627
                               alphasort);
 
2628
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
2629
#else   /* not __GLIBC__ */
 
2630
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
2631
                               alphasort);
 
2632
#endif  /* not __GLIBC__ */
 
2633
      if(numentries >= 0){
 
2634
        for(int i = 0; i < numentries; i++){
 
2635
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
 
2636
          if(ret == -1){
 
2637
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
 
2638
                         " \"%s\", 0): %s\n", tempdir,
 
2639
                         direntries[i]->d_name, strerror(errno));
 
2640
          }
 
2641
        }
 
2642
        
 
2643
        /* need to clean even if 0 because man page doesn't specify */
 
2644
        free(direntries);
 
2645
        if(numentries == -1){
 
2646
          perror_plus("scandir");
 
2647
        }
 
2648
        ret = rmdir(tempdir);
 
2649
        if(ret == -1 and errno != ENOENT){
 
2650
          perror_plus("rmdir");
 
2651
        }
2583
2652
      }
2584
 
    }
2585
 
 
2586
 
    /* need to clean even if 0 because man page doesn't specify */
2587
 
    free(direntries);
2588
 
    if(numentries == -1){
2589
 
      perror_plus("scandir");
2590
 
    }
2591
 
    ret = rmdir(tempdir);
2592
 
    if(ret == -1 and errno != ENOENT){
2593
 
      perror_plus("rmdir");
 
2653
      TEMP_FAILURE_RETRY(close(tempdir_fd));
2594
2654
    }
2595
2655
  }
2596
2656