/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-06-08 17:36:33 UTC
  • Revision ID: teddy@recompile.se-20140608173633-miqqavhoxym10uiz
Fix mandos server "--servicename" option, broken since 1.6.4.

* mandos (initlogger): Make "syslogger" a global variable again.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
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
1346
1347
    return 0;
1347
1348
  }
1348
1349
  
1349
 
  char *fullname = NULL;
1350
 
  ret = asprintf(&fullname, "%s/%s", hookdir, direntry->d_name);
1351
 
  if(ret < 0){
1352
 
    perror_plus("asprintf");
1353
 
    return 0;
1354
 
  }
1355
 
  
1356
 
  ret = stat(fullname, &st);
 
1350
  ret = fstatat(hookdir_fd, direntry->d_name, &st, 0);
1357
1351
  if(ret == -1){
1358
1352
    if(debug){
1359
1353
      perror_plus("Could not stat hook");
1360
1354
    }
1361
1355
    return 0;
1362
1356
  }
1363
 
  free(fullname);
1364
1357
  if(not (S_ISREG(st.st_mode))){
1365
1358
    /* Not a regular file */
1366
1359
    if(debug){
1514
1507
  return ret_errno;
1515
1508
}
1516
1509
 
1517
 
#ifndef O_CLOEXEC
1518
 
/*
1519
 
 * Based on the example in the GNU LibC manual chapter 13.13 "File
1520
 
 * Descriptor Flags".
1521
 
 | [[info:libc:Descriptor%20Flags][File Descriptor Flags]] |
1522
 
 */
1523
 
__attribute__((warn_unused_result))
1524
 
static int set_cloexec_flag(int fd){
1525
 
  int ret = (int)TEMP_FAILURE_RETRY(fcntl(fd, F_GETFD, 0));
1526
 
  /* If reading the flags failed, return error indication now. */
1527
 
  if(ret < 0){
1528
 
    return ret;
1529
 
  }
1530
 
  /* Store modified flag word in the descriptor. */
1531
 
  return (int)TEMP_FAILURE_RETRY(fcntl(fd, F_SETFD,
1532
 
                                       ret | FD_CLOEXEC));
1533
 
}
1534
 
#endif  /* not O_CLOEXEC */
1535
 
 
1536
1510
__attribute__((nonnull))
1537
1511
void run_network_hooks(const char *mode, const char *interface,
1538
1512
                       const float delay){
1539
1513
  struct dirent **direntries;
1540
1514
  if(hookdir_fd == -1){
1541
 
    hookdir_fd = open(hookdir, O_RDONLY |
1542
 
#ifdef O_CLOEXEC
1543
 
                      O_CLOEXEC
1544
 
#else  /* not O_CLOEXEC */
1545
 
                      0
1546
 
#endif  /* not O_CLOEXEC */
1547
 
                      );
 
1515
    hookdir_fd = open(hookdir, O_RDONLY);
1548
1516
    if(hookdir_fd == -1){
1549
1517
      if(errno == ENOENT){
1550
1518
        if(debug){
1556
1524
      }
1557
1525
      return;
1558
1526
    }
1559
 
#ifndef O_CLOEXEC
1560
 
    if(set_cloexec_flag(hookdir_fd) < 0){
1561
 
      perror_plus("set_cloexec_flag");
1562
 
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
1563
 
        perror_plus("close");
1564
 
      } else {
1565
 
        hookdir_fd = -1;
1566
 
      }
1567
 
      return;
1568
 
    }
1569
 
#endif  /* not O_CLOEXEC */
1570
1527
  }
1571
1528
#ifdef __GLIBC__
1572
1529
#if __GLIBC_PREREQ(2, 15)
1670
1627
          _exit(EX_OSERR);
1671
1628
        }
1672
1629
      }
1673
 
      if(fexecve(hookdir_fd, (char *const [])
1674
 
                 { direntry->d_name, NULL }, environ) == -1){
 
1630
      int hook_fd = openat(hookdir_fd, direntry->d_name, O_RDONLY);
 
1631
      if(hook_fd == -1){
 
1632
        perror_plus("openat");
 
1633
        _exit(EXIT_FAILURE);
 
1634
      }
 
1635
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
1636
        perror_plus("close");
 
1637
        _exit(EXIT_FAILURE);
 
1638
      }
 
1639
      if(fexecve(hook_fd, (char *const []){ direntry->d_name, NULL },
 
1640
                 environ) == -1){
1675
1641
        perror_plus("fexecve");
1676
1642
        _exit(EXIT_FAILURE);
1677
1643
      }
2625
2591
  /* Removes the GPGME temp directory and all files inside */
2626
2592
  if(tempdir != NULL){
2627
2593
    struct dirent **direntries = NULL;
2628
 
    struct dirent *direntry = NULL;
2629
 
    int numentries = scandir(tempdir, &direntries, notdotentries,
2630
 
                             alphasort);
2631
 
    if(numentries > 0){
2632
 
      for(int i = 0; i < numentries; i++){
2633
 
        direntry = direntries[i];
2634
 
        char *fullname = NULL;
2635
 
        ret = asprintf(&fullname, "%s/%s", tempdir,
2636
 
                       direntry->d_name);
2637
 
        if(ret < 0){
2638
 
          perror_plus("asprintf");
2639
 
          continue;
2640
 
        }
2641
 
        ret = remove(fullname);
2642
 
        if(ret == -1){
2643
 
          fprintf_plus(stderr, "remove(\"%s\"): %s\n", fullname,
2644
 
                       strerror(errno));
2645
 
        }
2646
 
        free(fullname);
 
2594
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY |
 
2595
                                                  O_NOFOLLOW));
 
2596
    if(tempdir_fd == -1){
 
2597
      perror_plus("open");
 
2598
    } else {
 
2599
#ifdef __GLIBC__
 
2600
#if __GLIBC_PREREQ(2, 15)
 
2601
      int numentries = scandirat(tempdir_fd, ".", &direntries,
 
2602
                                 notdotentries, alphasort);
 
2603
#else  /* not __GLIBC_PREREQ(2, 15) */
 
2604
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
2605
                               alphasort);
 
2606
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
2607
#else   /* not __GLIBC__ */
 
2608
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
2609
                               alphasort);
 
2610
#endif  /* not __GLIBC__ */
 
2611
      if(numentries > 0){
 
2612
        for(int i = 0; i < numentries; i++){
 
2613
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
 
2614
          if(ret == -1){
 
2615
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
 
2616
                         " \"%s\", 0): %s\n", tempdir,
 
2617
                         direntries[i]->d_name, strerror(errno));
 
2618
          }
 
2619
        }
 
2620
        
 
2621
        /* need to clean even if 0 because man page doesn't specify */
 
2622
        free(direntries);
 
2623
        if(numentries == -1){
 
2624
          perror_plus("scandir");
 
2625
        }
 
2626
        ret = rmdir(tempdir);
 
2627
        if(ret == -1 and errno != ENOENT){
 
2628
          perror_plus("rmdir");
 
2629
        }
2647
2630
      }
2648
 
    }
2649
 
    
2650
 
    /* need to clean even if 0 because man page doesn't specify */
2651
 
    free(direntries);
2652
 
    if(numentries == -1){
2653
 
      perror_plus("scandir");
2654
 
    }
2655
 
    ret = rmdir(tempdir);
2656
 
    if(ret == -1 and errno != ENOENT){
2657
 
      perror_plus("rmdir");
 
2631
      TEMP_FAILURE_RETRY(close(tempdir_fd));
2658
2632
    }
2659
2633
  }
2660
2634