/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: 2009-01-13 04:45:33 UTC
  • Revision ID: teddy@fukt.bsnet.se-20090113044533-de691yv0fa5w6quw
* plugins.d/mandos-client.c: Only comment changes.

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 <dirent.h>             /* opendir(), struct dirent, readdir()
 
59
                                 */
58
60
#include <inttypes.h>           /* PRIu16, SCNu16 */
59
61
#include <assert.h>             /* assert() */
60
62
#include <errno.h>              /* perror(), errno */
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'"
750
758
                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)))
864
872
        { .name = NULL }
865
873
      };
866
874
      
867
 
      error_t parse_opt (int key, char *arg,
868
 
                         struct argp_state *state) {
869
 
        switch (key) {
 
875
      error_t parse_opt(int key, char *arg,
 
876
                        struct argp_state *state) {
 
877
        switch(key) {
870
878
        case 128:               /* --debug */
871
879
          debug = true;
872
880
          break;
884
892
          break;
885
893
        case 129:               /* --dh-bits */
886
894
          ret = sscanf(arg, "%u", &mc.dh_bits);
887
 
          if(ret == 0 or mc.dh_bits == 0){
 
895
          if(ret != 1){
888
896
            fprintf(stderr, "Bad number of DH bits\n");
889
897
            exit(EXIT_FAILURE);
890
898
          }
893
901
          mc.priority = arg;
894
902
          break;
895
903
        case ARGP_KEY_ARG:
896
 
          argp_usage (state);
 
904
          argp_usage(state);
897
905
        case ARGP_KEY_END:
898
906
          break;
899
907
        default:
906
914
                           .args_doc = "",
907
915
                           .doc = "Mandos client -- Get and decrypt"
908
916
                           " passwords from a Mandos server" };
909
 
      ret = argp_parse (&argp, argc, argv, 0, 0, NULL);
910
 
      if (ret == ARGP_ERR_UNKNOWN){
 
917
      ret = argp_parse(&argp, argc, argv, 0, 0, NULL);
 
918
      if(ret == ARGP_ERR_UNKNOWN){
911
919
        fprintf(stderr, "Unknown error while parsing arguments\n");
912
920
        exitcode = EXIT_FAILURE;
913
921
        goto end;
948
956
    gid = getgid();
949
957
    
950
958
    ret = setuid(uid);
951
 
    if (ret == -1){
 
959
    if(ret == -1){
952
960
      perror("setuid");
953
961
    }
954
962
    
955
963
    setgid(gid);
956
 
    if (ret == -1){
 
964
    if(ret == -1){
957
965
      perror("setgid");
958
966
    }
959
967
    
960
968
    ret = init_gnutls_global(&mc, pubkey, seckey);
961
 
    if (ret == -1){
 
969
    if(ret == -1){
962
970
      fprintf(stderr, "init_gnutls_global failed\n");
963
971
      exitcode = EXIT_FAILURE;
964
972
      goto end;
997
1005
      }
998
1006
      uint16_t port;
999
1007
      ret = sscanf(address+1, "%" SCNu16, &port);
1000
 
      if(ret == 0 or port == 0){
 
1008
      if(ret != 1){
1001
1009
        fprintf(stderr, "Bad port number\n");
1002
1010
        exitcode = EXIT_FAILURE;
1003
1011
        goto end;
1013
1021
      goto end;
1014
1022
    }
1015
1023
    
1016
 
    if (not debug){
 
1024
    if(not debug){
1017
1025
      avahi_set_log_function(empty_log);
1018
1026
    }
1019
1027
    
1022
1030
    
1023
1031
    /* Allocate main Avahi loop object */
1024
1032
    mc.simple_poll = avahi_simple_poll_new();
1025
 
    if (mc.simple_poll == NULL) {
 
1033
    if(mc.simple_poll == NULL) {
1026
1034
        fprintf(stderr, "Avahi: Failed to create simple poll"
1027
1035
                " object.\n");
1028
1036
        exitcode = EXIT_FAILURE;
1048
1056
    }
1049
1057
    
1050
1058
    /* Check if creating the Avahi server object succeeded */
1051
 
    if (mc.server == NULL) {
 
1059
    if(mc.server == NULL) {
1052
1060
        fprintf(stderr, "Failed to create Avahi server: %s\n",
1053
1061
                avahi_strerror(error));
1054
1062
        exitcode = EXIT_FAILURE;
1060
1068
                                     AVAHI_PROTO_INET6,
1061
1069
                                     "_mandos._tcp", NULL, 0,
1062
1070
                                     browse_callback, &mc);
1063
 
    if (sb == NULL) {
 
1071
    if(sb == NULL) {
1064
1072
        fprintf(stderr, "Failed to create service browser: %s\n",
1065
1073
                avahi_strerror(avahi_server_errno(mc.server)));
1066
1074
        exitcode = EXIT_FAILURE;
1069
1077
    
1070
1078
    /* Run the main loop */
1071
1079
    
1072
 
    if (debug){
 
1080
    if(debug){
1073
1081
      fprintf(stderr, "Starting Avahi loop search\n");
1074
1082
    }
1075
1083
    
1077
1085
    
1078
1086
 end:
1079
1087
    
1080
 
    if (debug){
 
1088
    if(debug){
1081
1089
      fprintf(stderr, "%s exiting\n", argv[0]);
1082
1090
    }
1083
1091
    
1084
1092
    /* Cleanup things */
1085
 
    if (sb != NULL)
 
1093
    if(sb != NULL)
1086
1094
        avahi_s_service_browser_free(sb);
1087
1095
    
1088
 
    if (mc.server != NULL)
 
1096
    if(mc.server != NULL)
1089
1097
        avahi_server_free(mc.server);
1090
1098
    
1091
 
    if (mc.simple_poll != NULL)
 
1099
    if(mc.simple_poll != NULL)
1092
1100
        avahi_simple_poll_free(mc.simple_poll);
1093
1101
    
1094
 
    if (gnutls_initalized){
 
1102
    if(gnutls_initalized){
1095
1103
      gnutls_certificate_free_credentials(mc.cred);
1096
 
      gnutls_global_deinit ();
 
1104
      gnutls_global_deinit();
1097
1105
      gnutls_dh_params_deinit(mc.dh_params);
1098
1106
    }
1099
1107
    
1116
1124
          if(direntry == NULL){
1117
1125
            break;
1118
1126
          }
1119
 
          if (direntry->d_type == DT_REG){
 
1127
          if(direntry->d_type == DT_REG){
1120
1128
            char *fullname = NULL;
1121
1129
            ret = asprintf(&fullname, "%s/%s", tempdir,
1122
1130
                           direntry->d_name);