/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-03-23 19:24:40 UTC
  • Revision ID: teddy@recompile.se-20140323192440-d71iiqxebsxf9u2v
Update GCC warning flags and function attributes to GCC 4.7.

* Makefile (WARN): Update to include almost all warning flags.
* plugin-runner.c (getplugin, add_to_char_array, add_argument,
                   add_environment, set_cloexec_flag,
                   print_out_password): Add attribute
                                        "warn_unused_result".
  (main/parse_opt): Bug fix: Add error checking to --global-env,
                    --env-for, --plugin-dir, and --config-file, and
                    make sure errno does not "leak" from unrelated
                    functions.
* plugins.d/mandos-client.c
  (fprintf_plus, debuggnutls, resolve_callback): Add "nonnull"
                                                 attribute.
  (incbuffer, add_server, init_gpgme): Add "nonnull" and
                                       "warn_unused_result"
                                       attributes.
  (pgp_packet_decrypt, init_gnutls_global): - '' -
  (init_gnutls_session start_mandos_communication, get_flags): - '' -
  (good_flags, good_interface, interface_is_up): - '' -
  (interface_is_running, runnable_hook): - '' -
  (avahi_loop_with_timeout, bring_up_interface): : - '' -
  (safer_gnutls_strerror): Add "warn_unused_result" attribute.
  (notdotentries): Set "nonnull", "pure", and "warn_unused_result"
                   attributes.
  (raise_privileges, raise_privileges_permanently, lower_privileges,
  lower_privileges_permanently): Set "warn_unused_result" attribute.
  (run_network_hooks): Exit child process if it fails to do anything
                       it needs to do.  Make explicit cast to double
                       when passing float value to asprintf().  Change
                       return type to void - all callers changed.
  (bring_up_interface): Move variables "sd", "ret_errno", and
                        "ret_setflags" to innermost scope.  Bug fix:
                        Fail if could not get interface flags also in
                        non-debug mode, and restore old errno
                        correctly.  Print message if could not raise
                        (or later lower) privileges.
  (take_down_interface): Bug fix: When failing because it could not
                         get interface flags, restore old errno
                         correctly.  Print message if it could not
                         raise (or later lower) privileges.
  (main): Complain if failed to raise or lower privileges.  Only run
          network hooks or lower privileges if raising privileges was
          successful.

Show diffs side-by-side

added added

removed removed

Lines of Context:
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008-2014 Teddy Hogeborn
13
 
 * Copyright © 2008-2014 Björn Påhlsson
 
12
 * Copyright © 2008-2013 Teddy Hogeborn
 
13
 * Copyright © 2008-2013 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  /* not _LARGEFILE_SOURCE */
 
35
#endif
36
36
#ifndef _FILE_OFFSET_BITS
37
37
#define _FILE_OFFSET_BITS 64
38
 
#endif  /* not _FILE_OFFSET_BITS */
 
38
#endif
39
39
 
40
40
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), asprintf() */
41
41
 
42
42
#include <stdio.h>              /* fprintf(), stderr, fwrite(),
43
 
                                   stdout, ferror() */
 
43
                                   stdout, ferror(), remove() */
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(), unlinkat() */
 
60
#include <fcntl.h>              /* open() */
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(),
77
 
                                   unlinkat() */
 
76
                                   setgid(), pause(), _exit() */
78
77
#include <arpa/inet.h>          /* inet_pton(), htons() */
79
78
#include <iso646.h>             /* not, or, and */
80
79
#include <argp.h>               /* struct argp_option, error_t, struct
142
141
static const char sys_class_net[] = "/sys/class/net";
143
142
char *connect_to = NULL;
144
143
const char *hookdir = HOOKDIR;
145
 
int hookdir_fd = -1;
146
144
uid_t uid = 65534;
147
145
gid_t gid = 65534;
148
146
 
234
232
                          .af = af };
235
233
  if(new_server->ip == NULL){
236
234
    perror_plus("strdup");
237
 
    free(new_server);
238
235
    return false;
239
236
  }
240
237
  ret = clock_gettime(CLOCK_MONOTONIC, &(new_server->last_seen));
241
238
  if(ret == -1){
242
239
    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);
252
240
    return false;
253
241
  }
254
242
  /* Special case of first server */
270
258
 * Initialize GPGME.
271
259
 */
272
260
__attribute__((nonnull, warn_unused_result))
273
 
static bool init_gpgme(const char * const seckey,
274
 
                       const char * const pubkey,
275
 
                       const char * const tempdir,
276
 
                       mandos_context *mc){
 
261
static bool init_gpgme(const char *seckey, const char *pubkey,
 
262
                       const char *tempdir, mandos_context *mc){
277
263
  gpgme_error_t rc;
278
264
  gpgme_engine_info_t engine_info;
279
265
  
280
266
  /*
281
267
   * Helper function to insert pub and seckey to the engine keyring.
282
268
   */
283
 
  bool import_key(const char * const filename){
 
269
  bool import_key(const char *filename){
284
270
    int ret;
285
271
    int fd;
286
272
    gpgme_data_t pgp_data;
1076
1062
     timed out */
1077
1063
  
1078
1064
  if(quit_now){
1079
 
    avahi_s_service_resolver_free(r);
1080
1065
    return;
1081
1066
  }
1082
1067
  
1348
1333
  sret = strspn(direntry->d_name, "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
1349
1334
                "abcdefghijklmnopqrstuvwxyz"
1350
1335
                "0123456789"
1351
 
                "_.-");
 
1336
                "_-");
1352
1337
  if((direntry->d_name)[sret] != '\0'){
1353
1338
    /* Contains non-allowed characters */
1354
1339
    if(debug){
1358
1343
    return 0;
1359
1344
  }
1360
1345
  
1361
 
  ret = fstatat(hookdir_fd, direntry->d_name, &st, 0);
 
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);
1362
1354
  if(ret == -1){
1363
1355
    if(debug){
1364
1356
      perror_plus("Could not stat hook");
1469
1461
  error_t ret_errno = 0;
1470
1462
  if(seteuid(0) == -1){
1471
1463
    ret_errno = errno;
 
1464
    perror_plus("seteuid");
1472
1465
  }
1473
1466
  errno = old_errno;
1474
1467
  return ret_errno;
1485
1478
  }
1486
1479
  if(setuid(0) == -1){
1487
1480
    ret_errno = errno;
 
1481
    perror_plus("seteuid");
1488
1482
  }
1489
1483
  errno = old_errno;
1490
1484
  return ret_errno;
1497
1491
  error_t ret_errno = 0;
1498
1492
  if(seteuid(uid) == -1){
1499
1493
    ret_errno = errno;
 
1494
    perror_plus("seteuid");
1500
1495
  }
1501
1496
  errno = old_errno;
1502
1497
  return ret_errno;
1509
1504
  error_t ret_errno = 0;
1510
1505
  if(setuid(uid) == -1){
1511
1506
    ret_errno = errno;
 
1507
    perror_plus("setuid");
1512
1508
  }
1513
1509
  errno = old_errno;
1514
1510
  return ret_errno;
1517
1513
__attribute__((nonnull))
1518
1514
void run_network_hooks(const char *mode, const char *interface,
1519
1515
                       const float delay){
1520
 
  struct dirent **direntries = NULL;
1521
 
  if(hookdir_fd == -1){
1522
 
    hookdir_fd = open(hookdir, O_RDONLY);
1523
 
    if(hookdir_fd == -1){
1524
 
      if(errno == ENOENT){
1525
 
        if(debug){
1526
 
          fprintf_plus(stderr, "Network hook directory \"%s\" not"
1527
 
                       " found\n", hookdir);
1528
 
        }
1529
 
      } else {
1530
 
        perror_plus("open");
1531
 
      }
1532
 
      return;
1533
 
    }
1534
 
  }
1535
 
#ifdef __GLIBC__
1536
 
#if __GLIBC_PREREQ(2, 15)
1537
 
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1538
 
                           runnable_hook, alphasort);
1539
 
#else  /* not __GLIBC_PREREQ(2, 15) */
1540
 
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
1541
 
                         alphasort);
1542
 
#endif  /* not __GLIBC_PREREQ(2, 15) */
1543
 
#else   /* not __GLIBC__ */
1544
 
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
1545
 
                         alphasort);
1546
 
#endif  /* not __GLIBC__ */
 
1516
  struct dirent **direntries;
 
1517
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1518
                         alphasort);
1547
1519
  if(numhooks == -1){
1548
 
    perror_plus("scandir");
1549
 
    return;
1550
 
  }
1551
 
  struct dirent *direntry;
1552
 
  int ret;
1553
 
  int devnull = open("/dev/null", O_RDONLY);
1554
 
  for(int i = 0; i < numhooks; i++){
1555
 
    direntry = direntries[i];
1556
 
    if(debug){
1557
 
      fprintf_plus(stderr, "Running network hook \"%s\"\n",
1558
 
                   direntry->d_name);
 
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");
1559
1527
    }
1560
 
    pid_t hook_pid = fork();
1561
 
    if(hook_pid == 0){
1562
 
      /* Child */
1563
 
      /* Raise privileges */
1564
 
      errno = raise_privileges_permanently();
1565
 
      if(errno != 0){
1566
 
        perror_plus("Failed to raise privileges");
1567
 
        _exit(EX_NOPERM);
1568
 
      }
1569
 
      /* Set group */
1570
 
      errno = 0;
1571
 
      ret = setgid(0);
1572
 
      if(ret == -1){
1573
 
        perror_plus("setgid");
1574
 
        _exit(EX_NOPERM);
1575
 
      }
1576
 
      /* Reset supplementary groups */
1577
 
      errno = 0;
1578
 
      ret = setgroups(0, NULL);
1579
 
      if(ret == -1){
1580
 
        perror_plus("setgroups");
1581
 
        _exit(EX_NOPERM);
1582
 
      }
1583
 
      ret = dup2(devnull, STDIN_FILENO);
1584
 
      if(ret == -1){
1585
 
        perror_plus("dup2(devnull, STDIN_FILENO)");
1586
 
        _exit(EX_OSERR);
1587
 
      }
1588
 
      ret = close(devnull);
1589
 
      if(ret == -1){
1590
 
        perror_plus("close");
1591
 
        _exit(EX_OSERR);
1592
 
      }
1593
 
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
1594
 
      if(ret == -1){
1595
 
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
1596
 
        _exit(EX_OSERR);
1597
 
      }
1598
 
      ret = setenv("MANDOSNETHOOKDIR", hookdir, 1);
1599
 
      if(ret == -1){
1600
 
        perror_plus("setenv");
1601
 
        _exit(EX_OSERR);
1602
 
      }
1603
 
      ret = setenv("DEVICE", interface, 1);
1604
 
      if(ret == -1){
1605
 
        perror_plus("setenv");
1606
 
        _exit(EX_OSERR);
1607
 
      }
1608
 
      ret = setenv("VERBOSITY", debug ? "1" : "0", 1);
1609
 
      if(ret == -1){
1610
 
        perror_plus("setenv");
1611
 
        _exit(EX_OSERR);
1612
 
      }
1613
 
      ret = setenv("MODE", mode, 1);
1614
 
      if(ret == -1){
1615
 
        perror_plus("setenv");
1616
 
        _exit(EX_OSERR);
1617
 
      }
1618
 
      char *delaystring;
1619
 
      ret = asprintf(&delaystring, "%f", (double)delay);
1620
 
      if(ret == -1){
 
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){
1621
1537
        perror_plus("asprintf");
1622
 
        _exit(EX_OSERR);
1623
 
      }
1624
 
      ret = setenv("DELAY", delaystring, 1);
1625
 
      if(ret == -1){
 
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
        }
1626
1613
        free(delaystring);
1627
 
        perror_plus("setenv");
1628
 
        _exit(EX_OSERR);
1629
 
      }
1630
 
      free(delaystring);
1631
 
      if(connect_to != NULL){
1632
 
        ret = setenv("CONNECT", connect_to, 1);
1633
 
        if(ret == -1){
1634
 
          perror_plus("setenv");
1635
 
          _exit(EX_OSERR);
1636
 
        }
1637
 
      }
1638
 
      int hook_fd = openat(hookdir_fd, direntry->d_name, O_RDONLY);
1639
 
      if(hook_fd == -1){
1640
 
        perror_plus("openat");
1641
 
        _exit(EXIT_FAILURE);
1642
 
      }
1643
 
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
1644
 
        perror_plus("close");
1645
 
        _exit(EXIT_FAILURE);
1646
 
      }
1647
 
      if(fexecve(hook_fd, (char *const []){ direntry->d_name, NULL },
1648
 
                 environ) == -1){
1649
 
        perror_plus("fexecve");
1650
 
        _exit(EXIT_FAILURE);
1651
 
      }
1652
 
    } else {
1653
 
      if(hook_pid == -1){
1654
 
        perror_plus("fork");
1655
 
        free(direntry);
1656
 
        continue;
1657
 
      }
1658
 
      int status;
1659
 
      if(TEMP_FAILURE_RETRY(waitpid(hook_pid, &status, 0)) == -1){
1660
 
        perror_plus("waitpid");
1661
 
        free(direntry);
1662
 
        continue;
1663
 
      }
1664
 
      if(WIFEXITED(status)){
1665
 
        if(WEXITSTATUS(status) != 0){
1666
 
          fprintf_plus(stderr, "Warning: network hook \"%s\" exited"
1667
 
                       " with status %d\n", direntry->d_name,
1668
 
                       WEXITSTATUS(status));
1669
 
          free(direntry);
1670
 
          continue;
1671
 
        }
1672
 
      } else if(WIFSIGNALED(status)){
1673
 
        fprintf_plus(stderr, "Warning: network hook \"%s\" died by"
1674
 
                     " signal %d\n", direntry->d_name,
1675
 
                     WTERMSIG(status));
1676
 
        free(direntry);
1677
 
        continue;
 
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
        }
1678
1625
      } else {
1679
 
        fprintf_plus(stderr, "Warning: network hook \"%s\""
1680
 
                     " crashed\n", direntry->d_name);
1681
 
        free(direntry);
1682
 
        continue;
1683
 
      }
1684
 
    }
1685
 
    if(debug){
1686
 
      fprintf_plus(stderr, "Network hook \"%s\" ran successfully\n",
1687
 
                   direntry->d_name);
1688
 
    }
1689
 
    free(direntry);
1690
 
  }
1691
 
  free(direntries);
1692
 
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
1693
 
    perror_plus("close");
1694
 
  } else {
1695
 
    hookdir_fd = -1;
1696
 
  }
1697
 
  close(devnull);
 
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
  }
1698
1661
}
1699
1662
 
1700
1663
__attribute__((nonnull, warn_unused_result))
1750
1713
    
1751
1714
    /* Raise privileges */
1752
1715
    ret_errno = raise_privileges();
 
1716
    bool restore_loglevel = false;
1753
1717
    if(ret_errno != 0){
1754
 
      errno = ret_errno;
1755
1718
      perror_plus("Failed to raise privileges");
1756
1719
    }
1757
 
    
1758
1720
#ifdef __linux__
1759
1721
    int ret_linux;
1760
 
    bool restore_loglevel = false;
1761
1722
    if(ret_errno == 0){
1762
1723
      /* Lower kernel loglevel to KERN_NOTICE to avoid KERN_INFO
1763
1724
         messages about the network interface to mess up the prompt */
1861
1822
    /* Raise privileges */
1862
1823
    ret_errno = raise_privileges();
1863
1824
    if(ret_errno != 0){
1864
 
      errno = ret_errno;
1865
1825
      perror_plus("Failed to raise privileges");
1866
1826
    }
1867
 
    
1868
1827
    int ret_setflags = ioctl(sd, SIOCSIFFLAGS, &network);
1869
1828
    ioctl_errno = errno;
1870
1829
    
1912
1871
  int exitcode = EXIT_SUCCESS;
1913
1872
  char *interfaces_to_take_down = NULL;
1914
1873
  size_t interfaces_to_take_down_size = 0;
1915
 
  char run_tempdir[] = "/run/tmp/mandosXXXXXX";
1916
 
  char old_tempdir[] = "/tmp/mandosXXXXXX";
1917
 
  char *tempdir = NULL;
 
1874
  char tempdir[] = "/tmp/mandosXXXXXX";
 
1875
  bool tempdir_created = false;
1918
1876
  AvahiIfIndex if_index = AVAHI_IF_UNSPEC;
1919
1877
  const char *seckey = PATHDIR "/" SECKEY;
1920
1878
  const char *pubkey = PATHDIR "/" PUBKEY;
2274
2232
  
2275
2233
  /* If no interfaces were specified, make a list */
2276
2234
  if(mc.interfaces == NULL){
2277
 
    struct dirent **direntries = NULL;
 
2235
    struct dirent **direntries;
2278
2236
    /* Look for any good interfaces */
2279
2237
    ret = scandir(sys_class_net, &direntries, good_interface,
2280
2238
                  alphasort);
2286
2244
        if(ret_errno != 0){
2287
2245
          errno = ret_errno;
2288
2246
          perror_plus("argz_add");
2289
 
          free(direntries[i]);
2290
2247
          continue;
2291
2248
        }
2292
2249
        if(debug){
2293
2250
          fprintf_plus(stderr, "Will use interface \"%s\"\n",
2294
2251
                       direntries[i]->d_name);
2295
2252
        }
2296
 
        free(direntries[i]);
2297
2253
      }
2298
2254
      free(direntries);
2299
2255
    } else {
2300
 
      if(ret == 0){
2301
 
        free(direntries);
2302
 
      }
 
2256
      free(direntries);
2303
2257
      fprintf_plus(stderr, "Could not find a network interface\n");
2304
2258
      exitcode = EXIT_FAILURE;
2305
2259
      goto end;
2373
2327
    goto end;
2374
2328
  }
2375
2329
  
2376
 
  /* Try /run/tmp before /tmp */
2377
 
  tempdir = mkdtemp(run_tempdir);
2378
 
  if(tempdir == NULL and errno == ENOENT){
2379
 
      if(debug){
2380
 
        fprintf_plus(stderr, "Tempdir %s did not work, trying %s\n",
2381
 
                     run_tempdir, old_tempdir);
2382
 
      }
2383
 
      tempdir = mkdtemp(old_tempdir);
2384
 
  }
2385
 
  if(tempdir == NULL){
 
2330
  if(mkdtemp(tempdir) == NULL){
2386
2331
    perror_plus("mkdtemp");
2387
2332
    goto end;
2388
2333
  }
 
2334
  tempdir_created = true;
2389
2335
  
2390
2336
  if(quit_now){
2391
2337
    goto end;
2527
2473
  if(debug){
2528
2474
    fprintf_plus(stderr, "Starting Avahi loop search\n");
2529
2475
  }
2530
 
  
 
2476
 
2531
2477
  ret = avahi_loop_with_timeout(simple_poll,
2532
2478
                                (int)(retry_interval * 1000), &mc);
2533
2479
  if(debug){
2569
2515
    mc.current_server->prev->next = NULL;
2570
2516
    while(mc.current_server != NULL){
2571
2517
      server *next = mc.current_server->next;
2572
 
#ifdef __GNUC__
2573
 
#pragma GCC diagnostic push
2574
 
#pragma GCC diagnostic ignored "-Wcast-qual"
2575
 
#endif
2576
 
      free((char *)(mc.current_server->ip));
2577
 
#ifdef __GNUC__
2578
 
#pragma GCC diagnostic pop
2579
 
#endif
2580
2518
      free(mc.current_server);
2581
2519
      mc.current_server = next;
2582
2520
    }
2586
2524
  {
2587
2525
    ret_errno = raise_privileges();
2588
2526
    if(ret_errno != 0){
2589
 
      errno = ret_errno;
2590
2527
      perror_plus("Failed to raise privileges");
2591
2528
    } else {
2592
2529
      
2612
2549
        }
2613
2550
      }
2614
2551
    }
2615
 
    
2616
2552
    ret_errno = lower_privileges_permanently();
2617
2553
    if(ret_errno != 0){
2618
 
      errno = ret_errno;
2619
2554
      perror_plus("Failed to lower privileges permanently");
2620
2555
    }
2621
2556
  }
2624
2559
  free(interfaces_hooks);
2625
2560
  
2626
2561
  /* Removes the GPGME temp directory and all files inside */
2627
 
  if(tempdir != NULL){
 
2562
  if(tempdir_created){
2628
2563
    struct dirent **direntries = NULL;
2629
 
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY |
2630
 
                                                  O_NOFOLLOW));
2631
 
    if(tempdir_fd == -1){
2632
 
      perror_plus("open");
2633
 
    } else {
2634
 
#ifdef __GLIBC__
2635
 
#if __GLIBC_PREREQ(2, 15)
2636
 
      int numentries = scandirat(tempdir_fd, ".", &direntries,
2637
 
                                 notdotentries, alphasort);
2638
 
#else  /* not __GLIBC_PREREQ(2, 15) */
2639
 
      int numentries = scandir(tempdir, &direntries, notdotentries,
2640
 
                               alphasort);
2641
 
#endif  /* not __GLIBC_PREREQ(2, 15) */
2642
 
#else   /* not __GLIBC__ */
2643
 
      int numentries = scandir(tempdir, &direntries, notdotentries,
2644
 
                               alphasort);
2645
 
#endif  /* not __GLIBC__ */
2646
 
      if(numentries >= 0){
2647
 
        for(int i = 0; i < numentries; i++){
2648
 
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
2649
 
          if(ret == -1){
2650
 
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
2651
 
                         " \"%s\", 0): %s\n", tempdir,
2652
 
                         direntries[i]->d_name, strerror(errno));
2653
 
          }
2654
 
          free(direntries[i]);
2655
 
        }
2656
 
        
2657
 
        /* need to clean even if 0 because man page doesn't specify */
2658
 
        free(direntries);
2659
 
        if(numentries == -1){
2660
 
          perror_plus("scandir");
2661
 
        }
2662
 
        ret = rmdir(tempdir);
2663
 
        if(ret == -1 and errno != ENOENT){
2664
 
          perror_plus("rmdir");
2665
 
        }
 
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);
2666
2583
      }
2667
 
      TEMP_FAILURE_RETRY(close(tempdir_fd));
 
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");
2668
2594
    }
2669
2595
  }
2670
2596