/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

Merge from trunk; fix small memory leak in plugin-runner.

Show diffs side-by-side

added added

removed removed

Lines of Context:
48
48
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
49
49
                                   sockaddr_in6, PF_INET6,
50
50
                                   SOCK_STREAM, INET6_ADDRSTRLEN,
51
 
                                   uid_t, gid_t, open(), opendir(), DIR */
 
51
                                   uid_t, gid_t, open(), opendir(),
 
52
                                   DIR */
52
53
#include <sys/stat.h>           /* open() */
53
54
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
54
55
                                   struct in6_addr, inet_pton(),
55
56
                                   connect() */
56
57
#include <fcntl.h>              /* open() */
57
 
#include <dirent.h>             /* opendir(), struct dirent, readdir() */
58
 
#include <inttypes.h>           /* PRIu16, SCNu16 */
 
58
#include <dirent.h>             /* opendir(), struct dirent, readdir()
 
59
                                 */
 
60
#include <inttypes.h>           /* PRIu16, intmax_t, SCNdMAX */
59
61
#include <assert.h>             /* assert() */
60
62
#include <errno.h>              /* perror(), errno */
61
63
#include <time.h>               /* time() */
90
92
                                   gnutls_*
91
93
                                   init_gnutls_session(),
92
94
                                   GNUTLS_* */
93
 
#include <gnutls/openpgp.h>     /* gnutls_certificate_set_openpgp_key_file(),
 
95
#include <gnutls/openpgp.h>
 
96
                          /* gnutls_certificate_set_openpgp_key_file(),
94
97
                                   GNUTLS_OPENPGP_FMT_BASE64 */
95
98
 
96
99
/* GPGME */
129
132
 */
130
133
size_t adjustbuffer(char **buffer, size_t buffer_length,
131
134
                  size_t buffer_capacity){
132
 
  if (buffer_length + BUFFER_SIZE > buffer_capacity){
 
135
  if(buffer_length + BUFFER_SIZE > buffer_capacity){
133
136
    *buffer = realloc(*buffer, buffer_capacity + BUFFER_SIZE);
134
 
    if (buffer == NULL){
 
137
    if(buffer == NULL){
135
138
      return 0;
136
139
    }
137
140
    buffer_capacity += BUFFER_SIZE;
163
166
    }
164
167
    
165
168
    rc = gpgme_data_new_from_fd(&pgp_data, fd);
166
 
    if (rc != GPG_ERR_NO_ERROR){
 
169
    if(rc != GPG_ERR_NO_ERROR){
167
170
      fprintf(stderr, "bad gpgme_data_new_from_fd: %s: %s\n",
168
171
              gpgme_strsource(rc), gpgme_strerror(rc));
169
172
      return false;
170
173
    }
171
174
    
172
175
    rc = gpgme_op_import(mc->ctx, pgp_data);
173
 
    if (rc != GPG_ERR_NO_ERROR){
 
176
    if(rc != GPG_ERR_NO_ERROR){
174
177
      fprintf(stderr, "bad gpgme_op_import: %s: %s\n",
175
178
              gpgme_strsource(rc), gpgme_strerror(rc));
176
179
      return false;
184
187
    return true;
185
188
  }
186
189
  
187
 
  if (debug){
 
190
  if(debug){
188
191
    fprintf(stderr, "Initialize gpgme\n");
189
192
  }
190
193
  
191
194
  /* Init GPGME */
192
195
  gpgme_check_version(NULL);
193
196
  rc = gpgme_engine_check_version(GPGME_PROTOCOL_OpenPGP);
194
 
  if (rc != GPG_ERR_NO_ERROR){
 
197
  if(rc != GPG_ERR_NO_ERROR){
195
198
    fprintf(stderr, "bad gpgme_engine_check_version: %s: %s\n",
196
199
            gpgme_strsource(rc), gpgme_strerror(rc));
197
200
    return false;
198
201
  }
199
202
  
200
203
    /* Set GPGME home directory for the OpenPGP engine only */
201
 
  rc = gpgme_get_engine_info (&engine_info);
202
 
  if (rc != GPG_ERR_NO_ERROR){
 
204
  rc = gpgme_get_engine_info(&engine_info);
 
205
  if(rc != GPG_ERR_NO_ERROR){
203
206
    fprintf(stderr, "bad gpgme_get_engine_info: %s: %s\n",
204
207
            gpgme_strsource(rc), gpgme_strerror(rc));
205
208
    return false;
219
222
  
220
223
  /* Create new GPGME "context" */
221
224
  rc = gpgme_new(&(mc->ctx));
222
 
  if (rc != GPG_ERR_NO_ERROR){
 
225
  if(rc != GPG_ERR_NO_ERROR){
223
226
    fprintf(stderr, "bad gpgme_new: %s: %s\n",
224
227
            gpgme_strsource(rc), gpgme_strerror(rc));
225
228
    return false;
226
229
  }
227
230
  
228
 
  if (not import_key(pubkey) or not import_key(seckey)){
 
231
  if(not import_key(pubkey) or not import_key(seckey)){
229
232
    return false;
230
233
  }
231
234
  
236
239
 * Decrypt OpenPGP data.
237
240
 * Returns -1 on error
238
241
 */
239
 
static ssize_t pgp_packet_decrypt (const mandos_context *mc,
240
 
                                   const char *cryptotext,
241
 
                                   size_t crypto_size,
242
 
                                   char **plaintext){
 
242
static ssize_t pgp_packet_decrypt(const mandos_context *mc,
 
243
                                  const char *cryptotext,
 
244
                                  size_t crypto_size,
 
245
                                  char **plaintext){
243
246
  gpgme_data_t dh_crypto, dh_plain;
244
247
  gpgme_error_t rc;
245
248
  ssize_t ret;
246
249
  size_t plaintext_capacity = 0;
247
250
  ssize_t plaintext_length = 0;
248
251
  
249
 
  if (debug){
 
252
  if(debug){
250
253
    fprintf(stderr, "Trying to decrypt OpenPGP data\n");
251
254
  }
252
255
  
253
256
  /* Create new GPGME data buffer from memory cryptotext */
254
257
  rc = gpgme_data_new_from_mem(&dh_crypto, cryptotext, crypto_size,
255
258
                               0);
256
 
  if (rc != GPG_ERR_NO_ERROR){
 
259
  if(rc != GPG_ERR_NO_ERROR){
257
260
    fprintf(stderr, "bad gpgme_data_new_from_mem: %s: %s\n",
258
261
            gpgme_strsource(rc), gpgme_strerror(rc));
259
262
    return -1;
261
264
  
262
265
  /* Create new empty GPGME data buffer for the plaintext */
263
266
  rc = gpgme_data_new(&dh_plain);
264
 
  if (rc != GPG_ERR_NO_ERROR){
 
267
  if(rc != GPG_ERR_NO_ERROR){
265
268
    fprintf(stderr, "bad gpgme_data_new: %s: %s\n",
266
269
            gpgme_strsource(rc), gpgme_strerror(rc));
267
270
    gpgme_data_release(dh_crypto);
271
274
  /* Decrypt data from the cryptotext data buffer to the plaintext
272
275
     data buffer */
273
276
  rc = gpgme_op_decrypt(mc->ctx, dh_crypto, dh_plain);
274
 
  if (rc != GPG_ERR_NO_ERROR){
 
277
  if(rc != GPG_ERR_NO_ERROR){
275
278
    fprintf(stderr, "bad gpgme_op_decrypt: %s: %s\n",
276
279
            gpgme_strsource(rc), gpgme_strerror(rc));
277
280
    plaintext_length = -1;
278
 
    if (debug){
 
281
    if(debug){
279
282
      gpgme_decrypt_result_t result;
280
283
      result = gpgme_op_decrypt_result(mc->ctx);
281
 
      if (result == NULL){
 
284
      if(result == NULL){
282
285
        fprintf(stderr, "gpgme_op_decrypt_result failed\n");
283
286
      } else {
284
287
        fprintf(stderr, "Unsupported algorithm: %s\n",
311
314
  }
312
315
  
313
316
  /* Seek back to the beginning of the GPGME plaintext data buffer */
314
 
  if (gpgme_data_seek(dh_plain, (off_t) 0, SEEK_SET) == -1){
 
317
  if(gpgme_data_seek(dh_plain, (off_t)0, SEEK_SET) == -1){
315
318
    perror("gpgme_data_seek");
316
319
    plaintext_length = -1;
317
320
    goto decrypt_end;
322
325
    plaintext_capacity = adjustbuffer(plaintext,
323
326
                                      (size_t)plaintext_length,
324
327
                                      plaintext_capacity);
325
 
    if (plaintext_capacity == 0){
 
328
    if(plaintext_capacity == 0){
326
329
        perror("adjustbuffer");
327
330
        plaintext_length = -1;
328
331
        goto decrypt_end;
331
334
    ret = gpgme_data_read(dh_plain, *plaintext + plaintext_length,
332
335
                          BUFFER_SIZE);
333
336
    /* Print the data, if any */
334
 
    if (ret == 0){
 
337
    if(ret == 0){
335
338
      /* EOF */
336
339
      break;
337
340
    }
361
364
  return plaintext_length;
362
365
}
363
366
 
364
 
static const char * safer_gnutls_strerror (int value) {
365
 
  const char *ret = gnutls_strerror (value); /* Spurious warning */
366
 
  if (ret == NULL)
 
367
static const char * safer_gnutls_strerror(int value) {
 
368
  const char *ret = gnutls_strerror(value); /* Spurious warning from
 
369
                                               -Wunreachable-code */
 
370
  if(ret == NULL)
367
371
    ret = "(unknown)";
368
372
  return ret;
369
373
}
384
388
  }
385
389
  
386
390
  ret = gnutls_global_init();
387
 
  if (ret != GNUTLS_E_SUCCESS) {
388
 
    fprintf (stderr, "GnuTLS global_init: %s\n",
389
 
             safer_gnutls_strerror(ret));
 
391
  if(ret != GNUTLS_E_SUCCESS) {
 
392
    fprintf(stderr, "GnuTLS global_init: %s\n",
 
393
            safer_gnutls_strerror(ret));
390
394
    return -1;
391
395
  }
392
396
  
393
 
  if (debug){
 
397
  if(debug){
394
398
    /* "Use a log level over 10 to enable all debugging options."
395
399
     * - GnuTLS manual
396
400
     */
400
404
  
401
405
  /* OpenPGP credentials */
402
406
  gnutls_certificate_allocate_credentials(&mc->cred);
403
 
  if (ret != GNUTLS_E_SUCCESS){
404
 
    fprintf (stderr, "GnuTLS memory error: %s\n", /* Spurious
405
 
                                                     warning */
406
 
             safer_gnutls_strerror(ret));
407
 
    gnutls_global_deinit ();
 
407
  if(ret != GNUTLS_E_SUCCESS){
 
408
    fprintf(stderr, "GnuTLS memory error: %s\n", /* Spurious warning
 
409
                                                  * from
 
410
                                                  * -Wunreachable-code
 
411
                                                  */
 
412
            safer_gnutls_strerror(ret));
 
413
    gnutls_global_deinit();
408
414
    return -1;
409
415
  }
410
416
  
417
423
  ret = gnutls_certificate_set_openpgp_key_file
418
424
    (mc->cred, pubkeyfilename, seckeyfilename,
419
425
     GNUTLS_OPENPGP_FMT_BASE64);
420
 
  if (ret != GNUTLS_E_SUCCESS) {
 
426
  if(ret != GNUTLS_E_SUCCESS) {
421
427
    fprintf(stderr,
422
428
            "Error[%d] while reading the OpenPGP key pair ('%s',"
423
429
            " '%s')\n", ret, pubkeyfilename, seckeyfilename);
428
434
  
429
435
  /* GnuTLS server initialization */
430
436
  ret = gnutls_dh_params_init(&mc->dh_params);
431
 
  if (ret != GNUTLS_E_SUCCESS) {
432
 
    fprintf (stderr, "Error in GnuTLS DH parameter initialization:"
433
 
             " %s\n", safer_gnutls_strerror(ret));
 
437
  if(ret != GNUTLS_E_SUCCESS) {
 
438
    fprintf(stderr, "Error in GnuTLS DH parameter initialization:"
 
439
            " %s\n", safer_gnutls_strerror(ret));
434
440
    goto globalfail;
435
441
  }
436
442
  ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
437
 
  if (ret != GNUTLS_E_SUCCESS) {
438
 
    fprintf (stderr, "Error in GnuTLS prime generation: %s\n",
439
 
             safer_gnutls_strerror(ret));
 
443
  if(ret != GNUTLS_E_SUCCESS) {
 
444
    fprintf(stderr, "Error in GnuTLS prime generation: %s\n",
 
445
            safer_gnutls_strerror(ret));
440
446
    goto globalfail;
441
447
  }
442
448
  
457
463
  int ret;
458
464
  /* GnuTLS session creation */
459
465
  ret = gnutls_init(session, GNUTLS_SERVER);
460
 
  if (ret != GNUTLS_E_SUCCESS){
 
466
  if(ret != GNUTLS_E_SUCCESS){
461
467
    fprintf(stderr, "Error in GnuTLS session initialization: %s\n",
462
468
            safer_gnutls_strerror(ret));
463
469
  }
465
471
  {
466
472
    const char *err;
467
473
    ret = gnutls_priority_set_direct(*session, mc->priority, &err);
468
 
    if (ret != GNUTLS_E_SUCCESS) {
 
474
    if(ret != GNUTLS_E_SUCCESS) {
469
475
      fprintf(stderr, "Syntax error at: %s\n", err);
470
476
      fprintf(stderr, "GnuTLS error: %s\n",
471
477
              safer_gnutls_strerror(ret));
472
 
      gnutls_deinit (*session);
 
478
      gnutls_deinit(*session);
473
479
      return -1;
474
480
    }
475
481
  }
476
482
  
477
483
  ret = gnutls_credentials_set(*session, GNUTLS_CRD_CERTIFICATE,
478
484
                               mc->cred);
479
 
  if (ret != GNUTLS_E_SUCCESS) {
 
485
  if(ret != GNUTLS_E_SUCCESS) {
480
486
    fprintf(stderr, "Error setting GnuTLS credentials: %s\n",
481
487
            safer_gnutls_strerror(ret));
482
 
    gnutls_deinit (*session);
 
488
    gnutls_deinit(*session);
483
489
    return -1;
484
490
  }
485
491
  
486
492
  /* ignore client certificate if any. */
487
 
  gnutls_certificate_server_set_request (*session,
488
 
                                         GNUTLS_CERT_IGNORE);
 
493
  gnutls_certificate_server_set_request(*session,
 
494
                                        GNUTLS_CERT_IGNORE);
489
495
  
490
 
  gnutls_dh_set_prime_bits (*session, mc->dh_bits);
 
496
  gnutls_dh_set_prime_bits(*session, mc->dh_bits);
491
497
  
492
498
  return 0;
493
499
}
513
519
  char interface[IF_NAMESIZE];
514
520
  gnutls_session_t session;
515
521
  
516
 
  ret = init_gnutls_session (mc, &session);
517
 
  if (ret != 0){
 
522
  ret = init_gnutls_session(mc, &session);
 
523
  if(ret != 0){
518
524
    return -1;
519
525
  }
520
526
  
542
548
  /* It would be nice to have a way to detect if we were passed an
543
549
     IPv4 address here.   Now we assume an IPv6 address. */
544
550
  ret = inet_pton(AF_INET6, ip, &to.in6.sin6_addr);
545
 
  if (ret < 0 ){
 
551
  if(ret < 0 ){
546
552
    perror("inet_pton");
547
553
    return -1;
548
554
  }
550
556
    fprintf(stderr, "Bad address: %s\n", ip);
551
557
    return -1;
552
558
  }
553
 
  to.in6.sin6_port = htons(port); /* Spurious warning */
 
559
  to.in6.sin6_port = htons(port); /* Spurious warnings from
 
560
                                     -Wconversion and
 
561
                                     -Wunreachable-code */
554
562
  
555
563
  to.in6.sin6_scope_id = (uint32_t)if_index;
556
564
  
569
577
  }
570
578
  
571
579
  ret = connect(tcp_sd, &to.in, sizeof(to));
572
 
  if (ret < 0){
 
580
  if(ret < 0){
573
581
    perror("connect");
574
582
    return -1;
575
583
  }
576
584
  
577
585
  const char *out = mandos_protocol_version;
578
586
  written = 0;
579
 
  while (true){
 
587
  while(true){
580
588
    size_t out_size = strlen(out);
581
589
    ret = (int)TEMP_FAILURE_RETRY(write(tcp_sd, out + written,
582
590
                                   out_size - written));
583
 
    if (ret == -1){
 
591
    if(ret == -1){
584
592
      perror("write");
585
593
      retval = -1;
586
594
      goto mandos_end;
589
597
    if(written < out_size){
590
598
      continue;
591
599
    } else {
592
 
      if (out == mandos_protocol_version){
 
600
      if(out == mandos_protocol_version){
593
601
        written = 0;
594
602
        out = "\r\n";
595
603
      } else {
602
610
    fprintf(stderr, "Establishing TLS session with %s\n", ip);
603
611
  }
604
612
  
605
 
  gnutls_transport_set_ptr (session, (gnutls_transport_ptr_t) tcp_sd);
 
613
  gnutls_transport_set_ptr(session, (gnutls_transport_ptr_t) tcp_sd);
606
614
  
607
615
  do{
608
 
    ret = gnutls_handshake (session);
 
616
    ret = gnutls_handshake(session);
609
617
  } while(ret == GNUTLS_E_AGAIN or ret == GNUTLS_E_INTERRUPTED);
610
618
  
611
 
  if (ret != GNUTLS_E_SUCCESS){
 
619
  if(ret != GNUTLS_E_SUCCESS){
612
620
    if(debug){
613
621
      fprintf(stderr, "*** GnuTLS Handshake failed ***\n");
614
 
      gnutls_perror (ret);
 
622
      gnutls_perror(ret);
615
623
    }
616
624
    retval = -1;
617
625
    goto mandos_end;
627
635
  while(true){
628
636
    buffer_capacity = adjustbuffer(&buffer, buffer_length,
629
637
                                   buffer_capacity);
630
 
    if (buffer_capacity == 0){
 
638
    if(buffer_capacity == 0){
631
639
      perror("adjustbuffer");
632
640
      retval = -1;
633
641
      goto mandos_end;
635
643
    
636
644
    sret = gnutls_record_recv(session, buffer+buffer_length,
637
645
                              BUFFER_SIZE);
638
 
    if (sret == 0){
 
646
    if(sret == 0){
639
647
      break;
640
648
    }
641
 
    if (sret < 0){
 
649
    if(sret < 0){
642
650
      switch(sret){
643
651
      case GNUTLS_E_INTERRUPTED:
644
652
      case GNUTLS_E_AGAIN:
645
653
        break;
646
654
      case GNUTLS_E_REHANDSHAKE:
647
655
        do{
648
 
          ret = gnutls_handshake (session);
 
656
          ret = gnutls_handshake(session);
649
657
        } while(ret == GNUTLS_E_AGAIN or ret == GNUTLS_E_INTERRUPTED);
650
 
        if (ret < 0){
 
658
        if(ret < 0){
651
659
          fprintf(stderr, "*** GnuTLS Re-handshake failed ***\n");
652
 
          gnutls_perror (ret);
 
660
          gnutls_perror(ret);
653
661
          retval = -1;
654
662
          goto mandos_end;
655
663
        }
658
666
        fprintf(stderr, "Unknown error while reading data from"
659
667
                " encrypted session with Mandos server\n");
660
668
        retval = -1;
661
 
        gnutls_bye (session, GNUTLS_SHUT_RDWR);
 
669
        gnutls_bye(session, GNUTLS_SHUT_RDWR);
662
670
        goto mandos_end;
663
671
      }
664
672
    } else {
670
678
    fprintf(stderr, "Closing TLS session\n");
671
679
  }
672
680
  
673
 
  gnutls_bye (session, GNUTLS_SHUT_RDWR);
 
681
  gnutls_bye(session, GNUTLS_SHUT_RDWR);
674
682
  
675
 
  if (buffer_length > 0){
 
683
  if(buffer_length > 0){
676
684
    decrypted_buffer_size = pgp_packet_decrypt(mc, buffer,
677
685
                                               buffer_length,
678
686
                                               &decrypted_buffer);
679
 
    if (decrypted_buffer_size >= 0){
 
687
    if(decrypted_buffer_size >= 0){
680
688
      written = 0;
681
689
      while(written < (size_t) decrypted_buffer_size){
682
 
        ret = (int)fwrite (decrypted_buffer + written, 1,
683
 
                           (size_t)decrypted_buffer_size - written,
684
 
                           stdout);
 
690
        ret = (int)fwrite(decrypted_buffer + written, 1,
 
691
                          (size_t)decrypted_buffer_size - written,
 
692
                          stdout);
685
693
        if(ret == 0 and ferror(stdout)){
686
694
          if(debug){
687
695
            fprintf(stderr, "Error writing encrypted data: %s\n",
708
716
  if(ret == -1){
709
717
    perror("close");
710
718
  }
711
 
  gnutls_deinit (session);
 
719
  gnutls_deinit(session);
712
720
  return retval;
713
721
}
714
722
 
732
740
  /* Called whenever a service has been resolved successfully or
733
741
     timed out */
734
742
  
735
 
  switch (event) {
 
743
  switch(event) {
736
744
  default:
737
745
  case AVAHI_RESOLVER_FAILURE:
738
746
    fprintf(stderr, "(Avahi Resolver) Failed to resolve service '%s'"
746
754
      avahi_address_snprint(ip, sizeof(ip), address);
747
755
      if(debug){
748
756
        fprintf(stderr, "Mandos server \"%s\" found on %s (%s, %"
749
 
                PRIu16 ") on port %d\n", name, host_name, ip,
750
 
                interface, port);
 
757
                PRIdMAX ") on port %" PRIu16 "\n", name, host_name,
 
758
                ip, (intmax_t)interface, port);
751
759
      }
752
760
      int ret = start_mandos_communication(ip, port, interface, mc);
753
 
      if (ret == 0){
 
761
      if(ret == 0){
754
762
        avahi_simple_poll_quit(mc->simple_poll);
755
763
      }
756
764
    }
774
782
  /* Called whenever a new services becomes available on the LAN or
775
783
     is removed from the LAN */
776
784
  
777
 
  switch (event) {
 
785
  switch(event) {
778
786
  default:
779
787
  case AVAHI_BROWSER_FAILURE:
780
788
    
789
797
       the callback function is called the Avahi server will free the
790
798
       resolver for us. */
791
799
    
792
 
    if (!(avahi_s_service_resolver_new(mc->server, interface,
 
800
    if(!(avahi_s_service_resolver_new(mc->server, interface,
793
801
                                       protocol, name, type, domain,
794
802
                                       AVAHI_PROTO_INET6, 0,
795
803
                                       resolve_callback, mc)))
813
821
    AvahiSServiceBrowser *sb = NULL;
814
822
    int error;
815
823
    int ret;
 
824
    intmax_t tmpmax;
 
825
    int numchars;
816
826
    int exitcode = EXIT_SUCCESS;
817
827
    const char *interface = "eth0";
818
828
    struct ifreq network;
864
874
        { .name = NULL }
865
875
      };
866
876
      
867
 
      error_t parse_opt (int key, char *arg,
868
 
                         struct argp_state *state) {
869
 
        switch (key) {
 
877
      error_t parse_opt(int key, char *arg,
 
878
                        struct argp_state *state) {
 
879
        switch(key) {
870
880
        case 128:               /* --debug */
871
881
          debug = true;
872
882
          break;
883
893
          pubkey = arg;
884
894
          break;
885
895
        case 129:               /* --dh-bits */
886
 
          ret = sscanf(arg, "%u", &mc.dh_bits);
887
 
          if(ret == 0 or mc.dh_bits == 0){
 
896
          ret = sscanf(arg, "%" SCNdMAX "%n", &tmpmax, &numchars);
 
897
          if(ret < 1 or tmpmax != (typeof(mc.dh_bits))tmpmax
 
898
             or arg[numchars] != '\0'){
888
899
            fprintf(stderr, "Bad number of DH bits\n");
889
900
            exit(EXIT_FAILURE);
890
901
          }
 
902
          mc.dh_bits = (typeof(mc.dh_bits))tmpmax;
891
903
          break;
892
904
        case 130:               /* --priority */
893
905
          mc.priority = arg;
894
906
          break;
895
907
        case ARGP_KEY_ARG:
896
 
          argp_usage (state);
 
908
          argp_usage(state);
897
909
        case ARGP_KEY_END:
898
910
          break;
899
911
        default:
906
918
                           .args_doc = "",
907
919
                           .doc = "Mandos client -- Get and decrypt"
908
920
                           " passwords from a Mandos server" };
909
 
      ret = argp_parse (&argp, argc, argv, 0, 0, NULL);
910
 
      if (ret == ARGP_ERR_UNKNOWN){
 
921
      ret = argp_parse(&argp, argc, argv, 0, 0, NULL);
 
922
      if(ret == ARGP_ERR_UNKNOWN){
911
923
        fprintf(stderr, "Unknown error while parsing arguments\n");
912
924
        exitcode = EXIT_FAILURE;
913
925
        goto end;
948
960
    gid = getgid();
949
961
    
950
962
    ret = setuid(uid);
951
 
    if (ret == -1){
 
963
    if(ret == -1){
952
964
      perror("setuid");
953
965
    }
954
966
    
955
967
    setgid(gid);
956
 
    if (ret == -1){
 
968
    if(ret == -1){
957
969
      perror("setgid");
958
970
    }
959
971
    
960
972
    ret = init_gnutls_global(&mc, pubkey, seckey);
961
 
    if (ret == -1){
 
973
    if(ret == -1){
962
974
      fprintf(stderr, "init_gnutls_global failed\n");
963
975
      exitcode = EXIT_FAILURE;
964
976
      goto end;
996
1008
        goto end;
997
1009
      }
998
1010
      uint16_t port;
999
 
      ret = sscanf(address+1, "%" SCNu16, &port);
1000
 
      if(ret == 0 or port == 0){
 
1011
      ret = sscanf(address+1, "%" SCNdMAX "%n", &tmpmax, &numchars);
 
1012
      if(ret < 1 or tmpmax != (uint16_t)tmpmax
 
1013
         or address[numchars+1] != '\0'){
1001
1014
        fprintf(stderr, "Bad port number\n");
1002
1015
        exitcode = EXIT_FAILURE;
1003
1016
        goto end;
1004
1017
      }
 
1018
      port = (uint16_t)tmpmax;
1005
1019
      *address = '\0';
1006
1020
      address = connect_to;
1007
1021
      ret = start_mandos_communication(address, port, if_index, &mc);
1013
1027
      goto end;
1014
1028
    }
1015
1029
    
1016
 
    if (not debug){
 
1030
    if(not debug){
1017
1031
      avahi_set_log_function(empty_log);
1018
1032
    }
1019
1033
    
1022
1036
    
1023
1037
    /* Allocate main Avahi loop object */
1024
1038
    mc.simple_poll = avahi_simple_poll_new();
1025
 
    if (mc.simple_poll == NULL) {
 
1039
    if(mc.simple_poll == NULL) {
1026
1040
        fprintf(stderr, "Avahi: Failed to create simple poll"
1027
1041
                " object.\n");
1028
1042
        exitcode = EXIT_FAILURE;
1048
1062
    }
1049
1063
    
1050
1064
    /* Check if creating the Avahi server object succeeded */
1051
 
    if (mc.server == NULL) {
 
1065
    if(mc.server == NULL) {
1052
1066
        fprintf(stderr, "Failed to create Avahi server: %s\n",
1053
1067
                avahi_strerror(error));
1054
1068
        exitcode = EXIT_FAILURE;
1060
1074
                                     AVAHI_PROTO_INET6,
1061
1075
                                     "_mandos._tcp", NULL, 0,
1062
1076
                                     browse_callback, &mc);
1063
 
    if (sb == NULL) {
 
1077
    if(sb == NULL) {
1064
1078
        fprintf(stderr, "Failed to create service browser: %s\n",
1065
1079
                avahi_strerror(avahi_server_errno(mc.server)));
1066
1080
        exitcode = EXIT_FAILURE;
1069
1083
    
1070
1084
    /* Run the main loop */
1071
1085
    
1072
 
    if (debug){
 
1086
    if(debug){
1073
1087
      fprintf(stderr, "Starting Avahi loop search\n");
1074
1088
    }
1075
1089
    
1077
1091
    
1078
1092
 end:
1079
1093
    
1080
 
    if (debug){
 
1094
    if(debug){
1081
1095
      fprintf(stderr, "%s exiting\n", argv[0]);
1082
1096
    }
1083
1097
    
1084
1098
    /* Cleanup things */
1085
 
    if (sb != NULL)
 
1099
    if(sb != NULL)
1086
1100
        avahi_s_service_browser_free(sb);
1087
1101
    
1088
 
    if (mc.server != NULL)
 
1102
    if(mc.server != NULL)
1089
1103
        avahi_server_free(mc.server);
1090
1104
    
1091
 
    if (mc.simple_poll != NULL)
 
1105
    if(mc.simple_poll != NULL)
1092
1106
        avahi_simple_poll_free(mc.simple_poll);
1093
1107
    
1094
 
    if (gnutls_initalized){
 
1108
    if(gnutls_initalized){
1095
1109
      gnutls_certificate_free_credentials(mc.cred);
1096
 
      gnutls_global_deinit ();
 
1110
      gnutls_global_deinit();
1097
1111
      gnutls_dh_params_deinit(mc.dh_params);
1098
1112
    }
1099
1113
    
1116
1130
          if(direntry == NULL){
1117
1131
            break;
1118
1132
          }
1119
 
          if (direntry->d_type == DT_REG){
 
1133
          if(direntry->d_type == DT_REG){
1120
1134
            char *fullname = NULL;
1121
1135
            ret = asprintf(&fullname, "%s/%s", tempdir,
1122
1136
                           direntry->d_name);