/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: 2012-06-13 22:06:57 UTC
  • mto: This revision was merged to the branch mainline in revision 596.
  • Revision ID: teddy@recompile.se-20120613220657-qvq7c7nrndl3t413
* plugins.d/mandos-client.c (get_flags): Don't clobber errno.
  (up_interface): Removed; replaced with "interface_is_up".
  (interface_is_up, interface_is_running,
   lower_privileges_permanently, take_down_interface): New.
  (bring_up_interface): Return "error_t".  Use new functions
                        "interface_is_up", "get_flags", and
                        "interface_is_running".
  (main): Save all interfaces either autodetected or specified with
          --interface in argz vector "interfaces".  Save interfaces to
          take down on exit in argz vector "interfaces_to_take_down".
          Save interface names for DEVICE variable to network hooks as
          argz_vector "interfaces_hooks".  Bug fix: Be privileged
          while stopping network hooks.
* plugins.d/mandos-client.xml (SYNOPSIS): Changed --interface synopsis.
  (DESCRIPTION): Updated to document use of all interfaces.
  (OPTIONS): Updated description of "--interface".
* network-hooks.d/bridge: Parse comma-separated DEVICE environment
                          variable.
* network-hooks.d/openvpn: - '' -
* network-hooks.d/wireless: - '' -

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-2013 Teddy Hogeborn
13
 
 * Copyright © 2008-2013 Björn Påhlsson
 
12
 * Copyright © 2008-2012 Teddy Hogeborn
 
13
 * Copyright © 2008-2012 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
55
55
                                   opendir(), DIR */
56
56
#include <sys/stat.h>           /* open(), S_ISREG */
57
57
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
58
 
                                   inet_pton(), connect(),
59
 
                                   getnameinfo() */
 
58
                                   inet_pton(), connect() */
60
59
#include <fcntl.h>              /* open() */
61
60
#include <dirent.h>             /* opendir(), struct dirent, readdir()
62
61
                                 */
63
62
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
64
63
                                   strtoimax() */
 
64
#include <assert.h>             /* assert() */
65
65
#include <errno.h>              /* perror(), errno,
66
66
                                   program_invocation_short_name */
67
67
#include <time.h>               /* nanosleep(), time(), sleep() */
74
74
#include <unistd.h>             /* close(), SEEK_SET, off_t, write(),
75
75
                                   getuid(), getgid(), seteuid(),
76
76
                                   setgid(), pause(), _exit() */
77
 
#include <arpa/inet.h>          /* inet_pton(), htons() */
 
77
#include <arpa/inet.h>          /* inet_pton(), htons, inet_ntop() */
78
78
#include <iso646.h>             /* not, or, and */
79
79
#include <argp.h>               /* struct argp_option, error_t, struct
80
80
                                   argp_state, struct argp,
92
92
                                   argz_delete(), argz_append(),
93
93
                                   argz_stringify(), argz_add(),
94
94
                                   argz_count() */
95
 
#include <netdb.h>              /* getnameinfo(), NI_NUMERICHOST,
96
 
                                   EAI_SYSTEM, gai_strerror() */
97
95
 
98
96
#ifdef __linux__
99
97
#include <sys/klog.h>           /* klogctl() */
147
145
/* Doubly linked list that need to be circularly linked when used */
148
146
typedef struct server{
149
147
  const char *ip;
150
 
  in_port_t port;
 
148
  uint16_t port;
151
149
  AvahiIfIndex if_index;
152
150
  int af;
153
151
  struct timespec last_seen;
157
155
 
158
156
/* Used for passing in values through the Avahi callback functions */
159
157
typedef struct {
 
158
  AvahiSimplePoll *simple_poll;
160
159
  AvahiServer *server;
161
160
  gnutls_certificate_credentials_t cred;
162
161
  unsigned int dh_bits;
164
163
  const char *priority;
165
164
  gpgme_ctx_t ctx;
166
165
  server *current_server;
167
 
  char *interfaces;
168
 
  size_t interfaces_size;
169
166
} mandos_context;
170
167
 
171
 
/* global so signal handler can reach it*/
172
 
AvahiSimplePoll *simple_poll;
 
168
/* global context so signal handler can reach it*/
 
169
mandos_context mc = { .simple_poll = NULL, .server = NULL,
 
170
                      .dh_bits = 1024, .priority = "SECURE256"
 
171
                      ":!CTYPE-X.509:+CTYPE-OPENPGP",
 
172
                      .current_server = NULL };
173
173
 
174
174
sig_atomic_t quit_now = 0;
175
175
int signal_received = 0;
190
190
  
191
191
  TEMP_FAILURE_RETRY(fprintf(stream, "Mandos plugin %s: ",
192
192
                             program_invocation_short_name));
193
 
  return (int)TEMP_FAILURE_RETRY(vfprintf(stream, format, ap));
 
193
  return TEMP_FAILURE_RETRY(vfprintf(stream, format, ap));
194
194
}
195
195
 
196
196
/*
201
201
size_t incbuffer(char **buffer, size_t buffer_length,
202
202
                 size_t buffer_capacity){
203
203
  if(buffer_length + BUFFER_SIZE > buffer_capacity){
204
 
    char *new_buf = realloc(*buffer, buffer_capacity + BUFFER_SIZE);
205
 
    if(new_buf == NULL){
206
 
      int old_errno = errno;
207
 
      free(*buffer);
208
 
      errno = old_errno;
209
 
      *buffer = NULL;
 
204
    *buffer = realloc(*buffer, buffer_capacity + BUFFER_SIZE);
 
205
    if(buffer == NULL){
210
206
      return 0;
211
207
    }
212
 
    *buffer = new_buf;
213
208
    buffer_capacity += BUFFER_SIZE;
214
209
  }
215
210
  return buffer_capacity;
216
211
}
217
212
 
218
213
/* Add server to set of servers to retry periodically */
219
 
bool add_server(const char *ip, in_port_t port, AvahiIfIndex if_index,
220
 
                int af, server **current_server){
 
214
bool add_server(const char *ip, uint16_t port, AvahiIfIndex if_index,
 
215
                int af){
221
216
  int ret;
222
217
  server *new_server = malloc(sizeof(server));
223
218
  if(new_server == NULL){
232
227
    perror_plus("strdup");
233
228
    return false;
234
229
  }
235
 
  ret = clock_gettime(CLOCK_MONOTONIC, &(new_server->last_seen));
236
 
  if(ret == -1){
237
 
    perror_plus("clock_gettime");
238
 
    return false;
239
 
  }
240
230
  /* Special case of first server */
241
 
  if(*current_server == NULL){
 
231
  if (mc.current_server == NULL){
242
232
    new_server->next = new_server;
243
233
    new_server->prev = new_server;
244
 
    *current_server = new_server;
 
234
    mc.current_server = new_server;
245
235
  /* Place the new server last in the list */
246
236
  } else {
247
 
    new_server->next = *current_server;
248
 
    new_server->prev = (*current_server)->prev;
 
237
    new_server->next = mc.current_server;
 
238
    new_server->prev = mc.current_server->prev;
249
239
    new_server->prev->next = new_server;
250
 
    (*current_server)->prev = new_server;
 
240
    mc.current_server->prev = new_server;
 
241
  }
 
242
  ret = clock_gettime(CLOCK_MONOTONIC, &mc.current_server->last_seen);
 
243
  if(ret == -1){
 
244
    perror_plus("clock_gettime");
 
245
    return false;
251
246
  }
252
247
  return true;
253
248
}
256
251
 * Initialize GPGME.
257
252
 */
258
253
static bool init_gpgme(const char *seckey, const char *pubkey,
259
 
                       const char *tempdir, mandos_context *mc){
 
254
                       const char *tempdir){
260
255
  gpgme_error_t rc;
261
256
  gpgme_engine_info_t engine_info;
262
257
  
 
258
  
263
259
  /*
264
260
   * Helper function to insert pub and seckey to the engine keyring.
265
261
   */
281
277
      return false;
282
278
    }
283
279
    
284
 
    rc = gpgme_op_import(mc->ctx, pgp_data);
 
280
    rc = gpgme_op_import(mc.ctx, pgp_data);
285
281
    if(rc != GPG_ERR_NO_ERROR){
286
282
      fprintf_plus(stderr, "bad gpgme_op_import: %s: %s\n",
287
283
                   gpgme_strsource(rc), gpgme_strerror(rc));
331
327
  }
332
328
  
333
329
  /* Create new GPGME "context" */
334
 
  rc = gpgme_new(&(mc->ctx));
 
330
  rc = gpgme_new(&(mc.ctx));
335
331
  if(rc != GPG_ERR_NO_ERROR){
336
332
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
337
333
                 "bad gpgme_new: %s: %s\n", gpgme_strsource(rc),
352
348
 */
353
349
static ssize_t pgp_packet_decrypt(const char *cryptotext,
354
350
                                  size_t crypto_size,
355
 
                                  char **plaintext,
356
 
                                  mandos_context *mc){
 
351
                                  char **plaintext){
357
352
  gpgme_data_t dh_crypto, dh_plain;
358
353
  gpgme_error_t rc;
359
354
  ssize_t ret;
385
380
  
386
381
  /* Decrypt data from the cryptotext data buffer to the plaintext
387
382
     data buffer */
388
 
  rc = gpgme_op_decrypt(mc->ctx, dh_crypto, dh_plain);
 
383
  rc = gpgme_op_decrypt(mc.ctx, dh_crypto, dh_plain);
389
384
  if(rc != GPG_ERR_NO_ERROR){
390
385
    fprintf_plus(stderr, "bad gpgme_op_decrypt: %s: %s\n",
391
386
                 gpgme_strsource(rc), gpgme_strerror(rc));
392
387
    plaintext_length = -1;
393
388
    if(debug){
394
389
      gpgme_decrypt_result_t result;
395
 
      result = gpgme_op_decrypt_result(mc->ctx);
 
390
      result = gpgme_op_decrypt_result(mc.ctx);
396
391
      if(result == NULL){
397
392
        fprintf_plus(stderr, "gpgme_op_decrypt_result failed\n");
398
393
      } else {
476
471
}
477
472
 
478
473
static const char * safer_gnutls_strerror(int value){
479
 
  const char *ret = gnutls_strerror(value);
 
474
  const char *ret = gnutls_strerror(value); /* Spurious warning from
 
475
                                               -Wunreachable-code */
480
476
  if(ret == NULL)
481
477
    ret = "(unknown)";
482
478
  return ret;
489
485
}
490
486
 
491
487
static int init_gnutls_global(const char *pubkeyfilename,
492
 
                              const char *seckeyfilename,
493
 
                              mandos_context *mc){
 
488
                              const char *seckeyfilename){
494
489
  int ret;
495
490
  
496
491
  if(debug){
513
508
  }
514
509
  
515
510
  /* OpenPGP credentials */
516
 
  ret = gnutls_certificate_allocate_credentials(&mc->cred);
 
511
  ret = gnutls_certificate_allocate_credentials(&mc.cred);
517
512
  if(ret != GNUTLS_E_SUCCESS){
518
513
    fprintf_plus(stderr, "GnuTLS memory error: %s\n",
519
514
                 safer_gnutls_strerror(ret));
529
524
  }
530
525
  
531
526
  ret = gnutls_certificate_set_openpgp_key_file
532
 
    (mc->cred, pubkeyfilename, seckeyfilename,
 
527
    (mc.cred, pubkeyfilename, seckeyfilename,
533
528
     GNUTLS_OPENPGP_FMT_BASE64);
534
529
  if(ret != GNUTLS_E_SUCCESS){
535
530
    fprintf_plus(stderr,
541
536
  }
542
537
  
543
538
  /* GnuTLS server initialization */
544
 
  ret = gnutls_dh_params_init(&mc->dh_params);
 
539
  ret = gnutls_dh_params_init(&mc.dh_params);
545
540
  if(ret != GNUTLS_E_SUCCESS){
546
541
    fprintf_plus(stderr, "Error in GnuTLS DH parameter"
547
542
                 " initialization: %s\n",
548
543
                 safer_gnutls_strerror(ret));
549
544
    goto globalfail;
550
545
  }
551
 
  ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
 
546
  ret = gnutls_dh_params_generate2(mc.dh_params, mc.dh_bits);
552
547
  if(ret != GNUTLS_E_SUCCESS){
553
548
    fprintf_plus(stderr, "Error in GnuTLS prime generation: %s\n",
554
549
                 safer_gnutls_strerror(ret));
555
550
    goto globalfail;
556
551
  }
557
552
  
558
 
  gnutls_certificate_set_dh_params(mc->cred, mc->dh_params);
 
553
  gnutls_certificate_set_dh_params(mc.cred, mc.dh_params);
559
554
  
560
555
  return 0;
561
556
  
562
557
 globalfail:
563
558
  
564
 
  gnutls_certificate_free_credentials(mc->cred);
 
559
  gnutls_certificate_free_credentials(mc.cred);
565
560
  gnutls_global_deinit();
566
 
  gnutls_dh_params_deinit(mc->dh_params);
 
561
  gnutls_dh_params_deinit(mc.dh_params);
567
562
  return -1;
568
563
}
569
564
 
570
 
static int init_gnutls_session(gnutls_session_t *session,
571
 
                               mandos_context *mc){
 
565
static int init_gnutls_session(gnutls_session_t *session){
572
566
  int ret;
573
567
  /* GnuTLS session creation */
574
568
  do {
586
580
  {
587
581
    const char *err;
588
582
    do {
589
 
      ret = gnutls_priority_set_direct(*session, mc->priority, &err);
 
583
      ret = gnutls_priority_set_direct(*session, mc.priority, &err);
590
584
      if(quit_now){
591
585
        gnutls_deinit(*session);
592
586
        return -1;
603
597
  
604
598
  do {
605
599
    ret = gnutls_credentials_set(*session, GNUTLS_CRD_CERTIFICATE,
606
 
                                 mc->cred);
 
600
                                 mc.cred);
607
601
    if(quit_now){
608
602
      gnutls_deinit(*session);
609
603
      return -1;
619
613
  /* ignore client certificate if any. */
620
614
  gnutls_certificate_server_set_request(*session, GNUTLS_CERT_IGNORE);
621
615
  
622
 
  gnutls_dh_set_prime_bits(*session, mc->dh_bits);
 
616
  gnutls_dh_set_prime_bits(*session, mc.dh_bits);
623
617
  
624
618
  return 0;
625
619
}
629
623
                      __attribute__((unused)) const char *txt){}
630
624
 
631
625
/* Called when a Mandos server is found */
632
 
static int start_mandos_communication(const char *ip, in_port_t port,
 
626
static int start_mandos_communication(const char *ip, uint16_t port,
633
627
                                      AvahiIfIndex if_index,
634
 
                                      int af, mandos_context *mc){
 
628
                                      int af){
635
629
  int ret, tcp_sd = -1;
636
630
  ssize_t sret;
637
 
  struct sockaddr_storage to;
 
631
  union {
 
632
    struct sockaddr_in in;
 
633
    struct sockaddr_in6 in6;
 
634
  } to;
638
635
  char *buffer = NULL;
639
636
  char *decrypted_buffer = NULL;
640
637
  size_t buffer_length = 0;
664
661
    return -1;
665
662
  }
666
663
  
667
 
  /* If the interface is specified and we have a list of interfaces */
668
 
  if(if_index != AVAHI_IF_UNSPEC and mc->interfaces != NULL){
669
 
    /* Check if the interface is one of the interfaces we are using */
670
 
    bool match = false;
671
 
    {
672
 
      char *interface = NULL;
673
 
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
674
 
                                 interface))){
675
 
        if(if_nametoindex(interface) == (unsigned int)if_index){
676
 
          match = true;
677
 
          break;
678
 
        }
679
 
      }
680
 
    }
681
 
    if(not match){
682
 
      /* This interface does not match any in the list, so we don't
683
 
         connect to the server */
684
 
      if(debug){
685
 
        char interface[IF_NAMESIZE];
686
 
        if(if_indextoname((unsigned int)if_index, interface) == NULL){
687
 
          perror_plus("if_indextoname");
688
 
        } else {
689
 
          fprintf_plus(stderr, "Skipping server on non-used interface"
690
 
                       " \"%s\"\n",
691
 
                       if_indextoname((unsigned int)if_index,
692
 
                                      interface));
693
 
        }
694
 
      }
695
 
      return -1;
696
 
    }
697
 
  }
698
 
  
699
 
  ret = init_gnutls_session(&session, mc);
 
664
  ret = init_gnutls_session(&session);
700
665
  if(ret != 0){
701
666
    return -1;
702
667
  }
703
668
  
704
669
  if(debug){
705
670
    fprintf_plus(stderr, "Setting up a TCP connection to %s, port %"
706
 
                 PRIuMAX "\n", ip, (uintmax_t)port);
 
671
                 PRIu16 "\n", ip, port);
707
672
  }
708
673
  
709
674
  tcp_sd = socket(pf, SOCK_STREAM, 0);
721
686
  
722
687
  memset(&to, 0, sizeof(to));
723
688
  if(af == AF_INET6){
724
 
    ((struct sockaddr_in6 *)&to)->sin6_family = (sa_family_t)af;
725
 
    ret = inet_pton(af, ip, &((struct sockaddr_in6 *)&to)->sin6_addr);
 
689
    to.in6.sin6_family = (sa_family_t)af;
 
690
    ret = inet_pton(af, ip, &to.in6.sin6_addr);
726
691
  } else {                      /* IPv4 */
727
 
    ((struct sockaddr_in *)&to)->sin_family = (sa_family_t)af;
728
 
    ret = inet_pton(af, ip, &((struct sockaddr_in *)&to)->sin_addr);
 
692
    to.in.sin_family = (sa_family_t)af;
 
693
    ret = inet_pton(af, ip, &to.in.sin_addr);
729
694
  }
730
695
  if(ret < 0 ){
731
696
    int e = errno;
740
705
    goto mandos_end;
741
706
  }
742
707
  if(af == AF_INET6){
743
 
    ((struct sockaddr_in6 *)&to)->sin6_port = htons(port);    
744
 
    if(IN6_IS_ADDR_LINKLOCAL
745
 
       (&((struct sockaddr_in6 *)&to)->sin6_addr)){
 
708
    to.in6.sin6_port = htons(port); /* Spurious warnings from
 
709
                                       -Wconversion and
 
710
                                       -Wunreachable-code */
 
711
    
 
712
    if(IN6_IS_ADDR_LINKLOCAL /* Spurious warnings from */
 
713
       (&to.in6.sin6_addr)){ /* -Wstrict-aliasing=2 or lower and
 
714
                                -Wunreachable-code*/
746
715
      if(if_index == AVAHI_IF_UNSPEC){
747
716
        fprintf_plus(stderr, "An IPv6 link-local address is"
748
717
                     " incomplete without a network interface\n");
750
719
        goto mandos_end;
751
720
      }
752
721
      /* Set the network interface number as scope */
753
 
      ((struct sockaddr_in6 *)&to)->sin6_scope_id = (uint32_t)if_index;
 
722
      to.in6.sin6_scope_id = (uint32_t)if_index;
754
723
    }
755
724
  } else {
756
 
    ((struct sockaddr_in *)&to)->sin_port = htons(port);
 
725
    to.in.sin_port = htons(port); /* Spurious warnings from
 
726
                                     -Wconversion and
 
727
                                     -Wunreachable-code */
757
728
  }
758
729
  
759
730
  if(quit_now){
767
738
      if(if_indextoname((unsigned int)if_index, interface) == NULL){
768
739
        perror_plus("if_indextoname");
769
740
      } else {
770
 
        fprintf_plus(stderr, "Connection to: %s%%%s, port %" PRIuMAX
771
 
                     "\n", ip, interface, (uintmax_t)port);
 
741
        fprintf_plus(stderr, "Connection to: %s%%%s, port %" PRIu16
 
742
                     "\n", ip, interface, port);
772
743
      }
773
744
    } else {
774
 
      fprintf_plus(stderr, "Connection to: %s, port %" PRIuMAX "\n",
775
 
                   ip, (uintmax_t)port);
 
745
      fprintf_plus(stderr, "Connection to: %s, port %" PRIu16 "\n",
 
746
                   ip, port);
776
747
    }
777
748
    char addrstr[(INET_ADDRSTRLEN > INET6_ADDRSTRLEN) ?
778
749
                 INET_ADDRSTRLEN : INET6_ADDRSTRLEN] = "";
 
750
    const char *pcret;
779
751
    if(af == AF_INET6){
780
 
      ret = getnameinfo((struct sockaddr *)&to,
781
 
                        sizeof(struct sockaddr_in6),
782
 
                        addrstr, sizeof(addrstr), NULL, 0,
783
 
                        NI_NUMERICHOST);
 
752
      pcret = inet_ntop(af, &(to.in6.sin6_addr), addrstr,
 
753
                        sizeof(addrstr));
784
754
    } else {
785
 
      ret = getnameinfo((struct sockaddr *)&to,
786
 
                        sizeof(struct sockaddr_in),
787
 
                        addrstr, sizeof(addrstr), NULL, 0,
788
 
                        NI_NUMERICHOST);
 
755
      pcret = inet_ntop(af, &(to.in.sin_addr), addrstr,
 
756
                        sizeof(addrstr));
789
757
    }
790
 
    if(ret == EAI_SYSTEM){
791
 
      perror_plus("getnameinfo");
792
 
    } else if(ret != 0) {
793
 
      fprintf_plus(stderr, "getnameinfo: %s", gai_strerror(ret));
794
 
    } else if(strcmp(addrstr, ip) != 0){
795
 
      fprintf_plus(stderr, "Canonical address form: %s\n", addrstr);
 
758
    if(pcret == NULL){
 
759
      perror_plus("inet_ntop");
 
760
    } else {
 
761
      if(strcmp(addrstr, ip) != 0){
 
762
        fprintf_plus(stderr, "Canonical address form: %s\n", addrstr);
 
763
      }
796
764
    }
797
765
  }
798
766
  
802
770
  }
803
771
  
804
772
  if(af == AF_INET6){
805
 
    ret = connect(tcp_sd, (struct sockaddr *)&to,
806
 
                  sizeof(struct sockaddr_in6));
 
773
    ret = connect(tcp_sd, &to.in6, sizeof(to));
807
774
  } else {
808
 
    ret = connect(tcp_sd, (struct sockaddr *)&to, /* IPv4 */
809
 
                  sizeof(struct sockaddr_in));
 
775
    ret = connect(tcp_sd, &to.in, sizeof(to)); /* IPv4 */
810
776
  }
811
777
  if(ret < 0){
812
 
    if((errno != ECONNREFUSED and errno != ENETUNREACH) or debug){
 
778
    if ((errno != ECONNREFUSED and errno != ENETUNREACH) or debug){
813
779
      int e = errno;
814
780
      perror_plus("connect");
815
781
      errno = e;
976
942
  if(buffer_length > 0){
977
943
    ssize_t decrypted_buffer_size;
978
944
    decrypted_buffer_size = pgp_packet_decrypt(buffer, buffer_length,
979
 
                                               &decrypted_buffer, mc);
 
945
                                               &decrypted_buffer);
980
946
    if(decrypted_buffer_size >= 0){
981
947
      
982
948
      written = 0;
1043
1009
                             AVAHI_GCC_UNUSED AvahiStringList *txt,
1044
1010
                             AVAHI_GCC_UNUSED AvahiLookupResultFlags
1045
1011
                             flags,
1046
 
                             void* mc){
1047
 
  if(r == NULL){
1048
 
    return;
1049
 
  }
 
1012
                             AVAHI_GCC_UNUSED void* userdata){
 
1013
  assert(r);
1050
1014
  
1051
1015
  /* Called whenever a service has been resolved successfully or
1052
1016
     timed out */
1061
1025
    fprintf_plus(stderr, "(Avahi Resolver) Failed to resolve service "
1062
1026
                 "'%s' of type '%s' in domain '%s': %s\n", name, type,
1063
1027
                 domain,
1064
 
                 avahi_strerror(avahi_server_errno
1065
 
                                (((mandos_context*)mc)->server)));
 
1028
                 avahi_strerror(avahi_server_errno(mc.server)));
1066
1029
    break;
1067
1030
    
1068
1031
  case AVAHI_RESOLVER_FOUND:
1074
1037
                     PRIdMAX ") on port %" PRIu16 "\n", name,
1075
1038
                     host_name, ip, (intmax_t)interface, port);
1076
1039
      }
1077
 
      int ret = start_mandos_communication(ip, (in_port_t)port,
1078
 
                                           interface,
1079
 
                                           avahi_proto_to_af(proto),
1080
 
                                           mc);
 
1040
      int ret = start_mandos_communication(ip, port, interface,
 
1041
                                           avahi_proto_to_af(proto));
1081
1042
      if(ret == 0){
1082
 
        avahi_simple_poll_quit(simple_poll);
 
1043
        avahi_simple_poll_quit(mc.simple_poll);
1083
1044
      } else {
1084
 
        if(not add_server(ip, (in_port_t)port, interface,
1085
 
                          avahi_proto_to_af(proto),
1086
 
                          &((mandos_context*)mc)->current_server)){
 
1045
        if(not add_server(ip, port, interface,
 
1046
                          avahi_proto_to_af(proto))){
1087
1047
          fprintf_plus(stderr, "Failed to add server \"%s\" to server"
1088
1048
                       " list\n", name);
1089
1049
        }
1102
1062
                            const char *domain,
1103
1063
                            AVAHI_GCC_UNUSED AvahiLookupResultFlags
1104
1064
                            flags,
1105
 
                            void* mc){
1106
 
  if(b == NULL){
1107
 
    return;
1108
 
  }
 
1065
                            AVAHI_GCC_UNUSED void* userdata){
 
1066
  assert(b);
1109
1067
  
1110
1068
  /* Called whenever a new services becomes available on the LAN or
1111
1069
     is removed from the LAN */
1119
1077
  case AVAHI_BROWSER_FAILURE:
1120
1078
    
1121
1079
    fprintf_plus(stderr, "(Avahi browser) %s\n",
1122
 
                 avahi_strerror(avahi_server_errno
1123
 
                                (((mandos_context*)mc)->server)));
1124
 
    avahi_simple_poll_quit(simple_poll);
 
1080
                 avahi_strerror(avahi_server_errno(mc.server)));
 
1081
    avahi_simple_poll_quit(mc.simple_poll);
1125
1082
    return;
1126
1083
    
1127
1084
  case AVAHI_BROWSER_NEW:
1130
1087
       the callback function is called the Avahi server will free the
1131
1088
       resolver for us. */
1132
1089
    
1133
 
    if(avahi_s_service_resolver_new(((mandos_context*)mc)->server,
1134
 
                                    interface, protocol, name, type,
1135
 
                                    domain, protocol, 0,
1136
 
                                    resolve_callback, mc) == NULL)
 
1090
    if(avahi_s_service_resolver_new(mc.server, interface, protocol,
 
1091
                                    name, type, domain, protocol, 0,
 
1092
                                    resolve_callback, NULL) == NULL)
1137
1093
      fprintf_plus(stderr, "Avahi: Failed to resolve service '%s':"
1138
1094
                   " %s\n", name,
1139
 
                   avahi_strerror(avahi_server_errno
1140
 
                                  (((mandos_context*)mc)->server)));
 
1095
                   avahi_strerror(avahi_server_errno(mc.server)));
1141
1096
    break;
1142
1097
    
1143
1098
  case AVAHI_BROWSER_REMOVE:
1162
1117
  signal_received = sig;
1163
1118
  int old_errno = errno;
1164
1119
  /* set main loop to exit */
1165
 
  if(simple_poll != NULL){
1166
 
    avahi_simple_poll_quit(simple_poll);
 
1120
  if(mc.simple_poll != NULL){
 
1121
    avahi_simple_poll_quit(mc.simple_poll);
1167
1122
  }
1168
1123
  errno = old_errno;
1169
1124
}
1363
1318
  return 1;
1364
1319
}
1365
1320
 
1366
 
int avahi_loop_with_timeout(AvahiSimplePoll *s, int retry_interval,
1367
 
                            mandos_context *mc){
 
1321
int avahi_loop_with_timeout(AvahiSimplePoll *s, int retry_interval){
1368
1322
  int ret;
1369
1323
  struct timespec now;
1370
1324
  struct timespec waited_time;
1371
1325
  intmax_t block_time;
1372
1326
  
1373
1327
  while(true){
1374
 
    if(mc->current_server == NULL){
1375
 
      if(debug){
 
1328
    if(mc.current_server == NULL){
 
1329
      if (debug){
1376
1330
        fprintf_plus(stderr, "Wait until first server is found."
1377
1331
                     " No timeout!\n");
1378
1332
      }
1379
1333
      ret = avahi_simple_poll_iterate(s, -1);
1380
1334
    } else {
1381
 
      if(debug){
 
1335
      if (debug){
1382
1336
        fprintf_plus(stderr, "Check current_server if we should run"
1383
1337
                     " it, or wait\n");
1384
1338
      }
1391
1345
      /* Calculating in ms how long time between now and server
1392
1346
         who we visted longest time ago. Now - last seen.  */
1393
1347
      waited_time.tv_sec = (now.tv_sec
1394
 
                            - mc->current_server->last_seen.tv_sec);
 
1348
                            - mc.current_server->last_seen.tv_sec);
1395
1349
      waited_time.tv_nsec = (now.tv_nsec
1396
 
                             - mc->current_server->last_seen.tv_nsec);
 
1350
                             - mc.current_server->last_seen.tv_nsec);
1397
1351
      /* total time is 10s/10,000ms.
1398
1352
         Converting to s from ms by dividing by 1,000,
1399
1353
         and ns to ms by dividing by 1,000,000. */
1401
1355
                     - ((intmax_t)waited_time.tv_sec * 1000))
1402
1356
                    - ((intmax_t)waited_time.tv_nsec / 1000000));
1403
1357
      
1404
 
      if(debug){
 
1358
      if (debug){
1405
1359
        fprintf_plus(stderr, "Blocking for %" PRIdMAX " ms\n",
1406
1360
                     block_time);
1407
1361
      }
1408
1362
      
1409
1363
      if(block_time <= 0){
1410
 
        ret = start_mandos_communication(mc->current_server->ip,
1411
 
                                         mc->current_server->port,
1412
 
                                         mc->current_server->if_index,
1413
 
                                         mc->current_server->af, mc);
 
1364
        ret = start_mandos_communication(mc.current_server->ip,
 
1365
                                         mc.current_server->port,
 
1366
                                         mc.current_server->if_index,
 
1367
                                         mc.current_server->af);
1414
1368
        if(ret == 0){
1415
 
          avahi_simple_poll_quit(s);
 
1369
          avahi_simple_poll_quit(mc.simple_poll);
1416
1370
          return 0;
1417
1371
        }
1418
1372
        ret = clock_gettime(CLOCK_MONOTONIC,
1419
 
                            &mc->current_server->last_seen);
 
1373
                            &mc.current_server->last_seen);
1420
1374
        if(ret == -1){
1421
1375
          perror_plus("clock_gettime");
1422
1376
          return -1;
1423
1377
        }
1424
 
        mc->current_server = mc->current_server->next;
 
1378
        mc.current_server = mc.current_server->next;
1425
1379
        block_time = 0;         /* Call avahi to find new Mandos
1426
1380
                                   servers, but don't block */
1427
1381
      }
1429
1383
      ret = avahi_simple_poll_iterate(s, (int)block_time);
1430
1384
    }
1431
1385
    if(ret != 0){
1432
 
      if(ret > 0 or errno != EINTR){
 
1386
      if (ret > 0 or errno != EINTR){
1433
1387
        return (ret != 1) ? ret : 0;
1434
1388
      }
1435
1389
    }
1491
1445
bool run_network_hooks(const char *mode, const char *interface,
1492
1446
                       const float delay){
1493
1447
  struct dirent **direntries;
 
1448
  struct dirent *direntry;
 
1449
  int ret;
1494
1450
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
1495
1451
                         alphasort);
1496
1452
  if(numhooks == -1){
1497
 
    if(errno == ENOENT){
1498
 
      if(debug){
1499
 
        fprintf_plus(stderr, "Network hook directory \"%s\" not"
1500
 
                     " found\n", hookdir);
1501
 
      }
1502
 
    } else {
1503
 
      perror_plus("scandir");
1504
 
    }
 
1453
    perror_plus("scandir");
1505
1454
  } else {
1506
 
    struct dirent *direntry;
1507
 
    int ret;
1508
1455
    int devnull = open("/dev/null", O_RDONLY);
1509
1456
    for(int i = 0; i < numhooks; i++){
1510
1457
      direntry = direntries[i];
1730
1677
}
1731
1678
 
1732
1679
error_t take_down_interface(const char *const interface){
 
1680
  int sd = -1;
1733
1681
  error_t old_errno = errno;
 
1682
  error_t ret_errno = 0;
 
1683
  int ret, ret_setflags;
1734
1684
  struct ifreq network;
1735
1685
  unsigned int if_index = if_nametoindex(interface);
1736
1686
  if(if_index == 0){
1739
1689
    return ENXIO;
1740
1690
  }
1741
1691
  if(interface_is_up(interface)){
1742
 
    error_t ret_errno = 0;
1743
1692
    if(not get_flags(interface, &network) and debug){
1744
1693
      ret_errno = errno;
1745
1694
      fprintf_plus(stderr, "Failed to get flags for interface "
1748
1697
    }
1749
1698
    network.ifr_flags &= ~(short)IFF_UP; /* clear flag */
1750
1699
    
1751
 
    int sd = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
 
1700
    sd = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
1752
1701
    if(sd < 0){
1753
1702
      ret_errno = errno;
1754
1703
      perror_plus("socket");
1764
1713
    /* Raise priviliges */
1765
1714
    raise_privileges();
1766
1715
    
1767
 
    int ret_setflags = ioctl(sd, SIOCSIFFLAGS, &network);
 
1716
    ret_setflags = ioctl(sd, SIOCSIFFLAGS, &network);
1768
1717
    ret_errno = errno;
1769
1718
    
1770
1719
    /* Lower privileges */
1771
1720
    lower_privileges();
1772
1721
    
1773
1722
    /* Close the socket */
1774
 
    int ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
1723
    ret = (int)TEMP_FAILURE_RETRY(close(sd));
1775
1724
    if(ret == -1){
1776
1725
      perror_plus("close");
1777
1726
    }
1792
1741
}
1793
1742
 
1794
1743
int main(int argc, char *argv[]){
1795
 
  mandos_context mc = { .server = NULL, .dh_bits = 1024,
1796
 
                        .priority = "SECURE256:!CTYPE-X.509:"
1797
 
                        "+CTYPE-OPENPGP", .current_server = NULL, 
1798
 
                        .interfaces = NULL, .interfaces_size = 0 };
1799
1744
  AvahiSServiceBrowser *sb = NULL;
1800
1745
  error_t ret_errno;
1801
1746
  int ret;
1802
1747
  intmax_t tmpmax;
1803
1748
  char *tmp;
1804
1749
  int exitcode = EXIT_SUCCESS;
 
1750
  char *interfaces = NULL;
 
1751
  size_t interfaces_size = 0;
1805
1752
  char *interfaces_to_take_down = NULL;
1806
1753
  size_t interfaces_to_take_down_size = 0;
1807
1754
  char tempdir[] = "/tmp/mandosXXXXXX";
1810
1757
  const char *seckey = PATHDIR "/" SECKEY;
1811
1758
  const char *pubkey = PATHDIR "/" PUBKEY;
1812
1759
  char *interfaces_hooks = NULL;
 
1760
  size_t interfaces_hooks_size = 0;
1813
1761
  
1814
1762
  bool gnutls_initialized = false;
1815
1763
  bool gpgme_initialized = false;
1906
1854
        connect_to = arg;
1907
1855
        break;
1908
1856
      case 'i':                 /* --interface */
1909
 
        ret_errno = argz_add_sep(&mc.interfaces, &mc.interfaces_size,
1910
 
                                 arg, (int)',');
 
1857
        ret_errno = argz_add_sep(&interfaces, &interfaces_size, arg,
 
1858
                                 (int)',');
1911
1859
        if(ret_errno != 0){
1912
1860
          argp_error(state, "%s", strerror(ret_errno));
1913
1861
        }
2040
1988
      }
2041
1989
    
2042
1990
      /* Lower privileges */
2043
 
      lower_privileges();
 
1991
      errno = 0;
 
1992
      ret = seteuid(uid);
 
1993
      if(ret == -1){
 
1994
        perror_plus("seteuid");
 
1995
      }
2044
1996
    }
2045
1997
  }
2046
1998
  
2047
 
  /* Remove invalid interface names (except "none") */
 
1999
  /* Remove empty interface names */
2048
2000
  {
2049
2001
    char *interface = NULL;
2050
 
    while((interface = argz_next(mc.interfaces, mc.interfaces_size,
 
2002
    while((interface = argz_next(interfaces, interfaces_size,
2051
2003
                                 interface))){
2052
 
      if(strcmp(interface, "none") != 0
2053
 
         and if_nametoindex(interface) == 0){
2054
 
        if(interface[0] != '\0'){
 
2004
      if(if_nametoindex(interface) == 0){
 
2005
        if(interface[0] != '\0' and strcmp(interface, "none") != 0){
2055
2006
          fprintf_plus(stderr, "Not using nonexisting interface"
2056
2007
                       " \"%s\"\n", interface);
2057
2008
        }
2058
 
        argz_delete(&mc.interfaces, &mc.interfaces_size, interface);
 
2009
        argz_delete(&interfaces, &interfaces_size, interface);
2059
2010
        interface = NULL;
2060
2011
      }
2061
2012
    }
2063
2014
  
2064
2015
  /* Run network hooks */
2065
2016
  {
2066
 
    if(mc.interfaces != NULL){
2067
 
      interfaces_hooks = malloc(mc.interfaces_size);
2068
 
      if(interfaces_hooks == NULL){
2069
 
        perror_plus("malloc");
2070
 
        goto end;
2071
 
      }
2072
 
      memcpy(interfaces_hooks, mc.interfaces, mc.interfaces_size);
2073
 
      argz_stringify(interfaces_hooks, mc.interfaces_size, (int)',');
 
2017
    ret_errno = argz_append(&interfaces_hooks, &interfaces_hooks_size,
 
2018
                            interfaces, interfaces_size);
 
2019
    if(ret_errno != 0){
 
2020
      errno = ret_errno;
 
2021
      perror_plus("argz_append");
 
2022
      goto end;
2074
2023
    }
2075
 
    if(not run_network_hooks("start", interfaces_hooks != NULL ?
2076
 
                             interfaces_hooks : "", delay)){
 
2024
    argz_stringify(interfaces_hooks, interfaces_hooks_size, (int)',');
 
2025
    if(not run_network_hooks("start", interfaces_hooks, delay)){
2077
2026
      goto end;
2078
2027
    }
2079
2028
  }
2086
2035
     from the signal handler */
2087
2036
  /* Initialize the pseudo-RNG for Avahi */
2088
2037
  srand((unsigned int) time(NULL));
2089
 
  simple_poll = avahi_simple_poll_new();
2090
 
  if(simple_poll == NULL){
 
2038
  mc.simple_poll = avahi_simple_poll_new();
 
2039
  if(mc.simple_poll == NULL){
2091
2040
    fprintf_plus(stderr,
2092
2041
                 "Avahi: Failed to create simple poll object.\n");
2093
2042
    exitcode = EX_UNAVAILABLE;
2158
2107
  }
2159
2108
  
2160
2109
  /* If no interfaces were specified, make a list */
2161
 
  if(mc.interfaces == NULL){
 
2110
  if(interfaces == NULL){
2162
2111
    struct dirent **direntries;
2163
2112
    /* Look for any good interfaces */
2164
2113
    ret = scandir(sys_class_net, &direntries, good_interface,
2166
2115
    if(ret >= 1){
2167
2116
      /* Add all found interfaces to interfaces list */
2168
2117
      for(int i = 0; i < ret; ++i){
2169
 
        ret_errno = argz_add(&mc.interfaces, &mc.interfaces_size,
 
2118
        ret_errno = argz_add(&interfaces, &interfaces_size,
2170
2119
                             direntries[i]->d_name);
2171
2120
        if(ret_errno != 0){
2172
 
          errno = ret_errno;
2173
2121
          perror_plus("argz_add");
2174
2122
          continue;
2175
2123
        }
2187
2135
    }
2188
2136
  }
2189
2137
  
2190
 
  /* Bring up interfaces which are down, and remove any "none"s */
2191
 
  {
 
2138
  /* If we only got one interface, explicitly use only that one */
 
2139
  if(argz_count(interfaces, interfaces_size) == 1){
 
2140
    if(debug){
 
2141
      fprintf_plus(stderr, "Using only interface \"%s\"\n",
 
2142
                   interfaces);
 
2143
    }
 
2144
    if_index = (AvahiIfIndex)if_nametoindex(interfaces);
 
2145
  }
 
2146
  
 
2147
  /* Bring up interfaces which are down */
 
2148
  if(not (argz_count(interfaces, interfaces_size) == 1
 
2149
          and strcmp(interfaces, "none") == 0)){
2192
2150
    char *interface = NULL;
2193
 
    while((interface = argz_next(mc.interfaces, mc.interfaces_size,
 
2151
    while((interface = argz_next(interfaces, interfaces_size,
2194
2152
                                 interface))){
2195
 
      /* If interface name is "none", stop bringing up interfaces.
2196
 
         Also remove all instances of "none" from the list */
2197
 
      if(strcmp(interface, "none") == 0){
2198
 
        argz_delete(&mc.interfaces, &mc.interfaces_size,
2199
 
                    interface);
2200
 
        interface = NULL;
2201
 
        while((interface = argz_next(mc.interfaces,
2202
 
                                     mc.interfaces_size, interface))){
2203
 
          if(strcmp(interface, "none") == 0){
2204
 
            argz_delete(&mc.interfaces, &mc.interfaces_size,
2205
 
                        interface);
2206
 
            interface = NULL;
2207
 
          }
2208
 
        }
2209
 
        break;
2210
 
      }
2211
2153
      bool interface_was_up = interface_is_up(interface);
2212
2154
      ret = bring_up_interface(interface, delay);
2213
2155
      if(not interface_was_up){
2218
2160
          ret_errno = argz_add(&interfaces_to_take_down,
2219
2161
                               &interfaces_to_take_down_size,
2220
2162
                               interface);
2221
 
          if(ret_errno != 0){
2222
 
            errno = ret_errno;
2223
 
            perror_plus("argz_add");
2224
 
          }
2225
2163
        }
2226
2164
      }
2227
2165
    }
 
2166
    free(interfaces);
 
2167
    interfaces = NULL;
 
2168
    interfaces_size = 0;
2228
2169
    if(debug and (interfaces_to_take_down == NULL)){
2229
2170
      fprintf_plus(stderr, "No interfaces were brought up\n");
2230
2171
    }
2231
2172
  }
2232
2173
  
2233
 
  /* If we only got one interface, explicitly use only that one */
2234
 
  if(argz_count(mc.interfaces, mc.interfaces_size) == 1){
2235
 
    if(debug){
2236
 
      fprintf_plus(stderr, "Using only interface \"%s\"\n",
2237
 
                   mc.interfaces);
2238
 
    }
2239
 
    if_index = (AvahiIfIndex)if_nametoindex(mc.interfaces);
2240
 
  }
2241
 
  
2242
2174
  if(quit_now){
2243
2175
    goto end;
2244
2176
  }
2245
2177
  
2246
 
  ret = init_gnutls_global(pubkey, seckey, &mc);
 
2178
  ret = init_gnutls_global(pubkey, seckey);
2247
2179
  if(ret == -1){
2248
2180
    fprintf_plus(stderr, "init_gnutls_global failed\n");
2249
2181
    exitcode = EX_UNAVAILABLE;
2266
2198
    goto end;
2267
2199
  }
2268
2200
  
2269
 
  if(not init_gpgme(pubkey, seckey, tempdir, &mc)){
 
2201
  if(not init_gpgme(pubkey, seckey, tempdir)){
2270
2202
    fprintf_plus(stderr, "init_gpgme failed\n");
2271
2203
    exitcode = EX_UNAVAILABLE;
2272
2204
    goto end;
2293
2225
      goto end;
2294
2226
    }
2295
2227
    
2296
 
    in_port_t port;
 
2228
    uint16_t port;
2297
2229
    errno = 0;
2298
2230
    tmpmax = strtoimax(address+1, &tmp, 10);
2299
2231
    if(errno != 0 or tmp == address+1 or *tmp != '\0'
2300
 
       or tmpmax != (in_port_t)tmpmax){
 
2232
       or tmpmax != (uint16_t)tmpmax){
2301
2233
      fprintf_plus(stderr, "Bad port number\n");
2302
2234
      exitcode = EX_USAGE;
2303
2235
      goto end;
2304
2236
    }
2305
 
    
 
2237
  
2306
2238
    if(quit_now){
2307
2239
      goto end;
2308
2240
    }
2309
2241
    
2310
 
    port = (in_port_t)tmpmax;
 
2242
    port = (uint16_t)tmpmax;
2311
2243
    *address = '\0';
2312
2244
    /* Colon in address indicates IPv6 */
2313
2245
    int af;
2329
2261
    }
2330
2262
    
2331
2263
    while(not quit_now){
2332
 
      ret = start_mandos_communication(address, port, if_index, af,
2333
 
                                       &mc);
 
2264
      ret = start_mandos_communication(address, port, if_index, af);
2334
2265
      if(quit_now or ret == 0){
2335
2266
        break;
2336
2267
      }
2338
2269
        fprintf_plus(stderr, "Retrying in %d seconds\n",
2339
2270
                     (int)retry_interval);
2340
2271
      }
2341
 
      sleep((unsigned int)retry_interval);
 
2272
      sleep((int)retry_interval);
2342
2273
    }
2343
2274
    
2344
 
    if(not quit_now){
 
2275
    if (not quit_now){
2345
2276
      exitcode = EXIT_SUCCESS;
2346
2277
    }
2347
2278
    
2362
2293
    config.publish_domain = 0;
2363
2294
    
2364
2295
    /* Allocate a new server */
2365
 
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
2366
 
                                 &config, NULL, NULL, &ret_errno);
 
2296
    mc.server = avahi_server_new(avahi_simple_poll_get
 
2297
                                 (mc.simple_poll), &config, NULL,
 
2298
                                 NULL, &ret_errno);
2367
2299
    
2368
2300
    /* Free the Avahi configuration data */
2369
2301
    avahi_server_config_free(&config);
2384
2316
  /* Create the Avahi service browser */
2385
2317
  sb = avahi_s_service_browser_new(mc.server, if_index,
2386
2318
                                   AVAHI_PROTO_UNSPEC, "_mandos._tcp",
2387
 
                                   NULL, 0, browse_callback,
2388
 
                                   (void *)&mc);
 
2319
                                   NULL, 0, browse_callback, NULL);
2389
2320
  if(sb == NULL){
2390
2321
    fprintf_plus(stderr, "Failed to create service browser: %s\n",
2391
2322
                 avahi_strerror(avahi_server_errno(mc.server)));
2403
2334
    fprintf_plus(stderr, "Starting Avahi loop search\n");
2404
2335
  }
2405
2336
 
2406
 
  ret = avahi_loop_with_timeout(simple_poll,
2407
 
                                (int)(retry_interval * 1000), &mc);
 
2337
  ret = avahi_loop_with_timeout(mc.simple_poll,
 
2338
                                (int)(retry_interval * 1000));
2408
2339
  if(debug){
2409
2340
    fprintf_plus(stderr, "avahi_loop_with_timeout exited %s\n",
2410
2341
                 (ret == 0) ? "successfully" : "with error");
2417
2348
  }
2418
2349
  
2419
2350
  /* Cleanup things */
2420
 
  free(mc.interfaces);
2421
 
  
2422
2351
  if(sb != NULL)
2423
2352
    avahi_s_service_browser_free(sb);
2424
2353
  
2425
2354
  if(mc.server != NULL)
2426
2355
    avahi_server_free(mc.server);
2427
2356
  
2428
 
  if(simple_poll != NULL)
2429
 
    avahi_simple_poll_free(simple_poll);
 
2357
  if(mc.simple_poll != NULL)
 
2358
    avahi_simple_poll_free(mc.simple_poll);
2430
2359
  
2431
2360
  if(gnutls_initialized){
2432
2361
    gnutls_certificate_free_credentials(mc.cred);
2454
2383
    raise_privileges();
2455
2384
    
2456
2385
    /* Run network hooks */
2457
 
    run_network_hooks("stop", interfaces_hooks != NULL ?
2458
 
                      interfaces_hooks : "", delay);
 
2386
    run_network_hooks("stop", interfaces_hooks, delay);
2459
2387
    
2460
2388
    /* Take down the network interfaces which were brought up */
2461
2389
    {
2487
2415
    struct dirent *direntry = NULL;
2488
2416
    int numentries = scandir(tempdir, &direntries, notdotentries,
2489
2417
                             alphasort);
2490
 
    if(numentries > 0){
 
2418
    if (numentries > 0){
2491
2419
      for(int i = 0; i < numentries; i++){
2492
2420
        direntry = direntries[i];
2493
2421
        char *fullname = NULL;
2508
2436
 
2509
2437
    /* need to clean even if 0 because man page doesn't specify */
2510
2438
    free(direntries);
2511
 
    if(numentries == -1){
 
2439
    if (numentries == -1){
2512
2440
      perror_plus("scandir");
2513
2441
    }
2514
2442
    ret = rmdir(tempdir);