/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: 2018-02-18 01:03:48 UTC
  • Revision ID: teddy@recompile.se-20180218010348-hhjfox5t4py5n5zh
Client: Bug fix: Possible memory leak if failing to open /dev/null

* plugins.d/mandos-client.d (run_network_hooks): Try to open /dev/null
  before reading network hook dir in order to fail early in that case.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
123
123
                                   gnutls_*
124
124
                                   init_gnutls_session(),
125
125
                                   GNUTLS_* */
126
 
#if GNUTLS_VERSION_NUMBER < 0x030600
127
126
#include <gnutls/openpgp.h>
128
127
                         /* gnutls_certificate_set_openpgp_key_file(),
129
128
                            GNUTLS_OPENPGP_FMT_BASE64 */
130
 
#elif GNUTLS_VERSION_NUMBER >= 0x030606
131
 
#include <gnutls/x509.h>        /* gnutls_pkcs_encrypt_flags_t,
132
 
                                 GNUTLS_PKCS_PLAIN,
133
 
                                 GNUTLS_PKCS_NULL_PASSWORD */
134
 
#endif
135
129
 
136
130
/* GPGME */
137
131
#include <gpgme.h>              /* All GPGME types, constants and
145
139
#define PATHDIR "/conf/conf.d/mandos"
146
140
#define SECKEY "seckey.txt"
147
141
#define PUBKEY "pubkey.txt"
148
 
#define TLS_PRIVKEY "tls-privkey.pem"
149
 
#define TLS_PUBKEY "tls-pubkey.pem"
150
142
#define HOOKDIR "/lib/mandos/network-hooks.d"
151
143
 
152
144
bool debug = false;
280
272
  return true;
281
273
}
282
274
 
283
 
/* Set effective uid to 0, return errno */
284
 
__attribute__((warn_unused_result))
285
 
int raise_privileges(void){
286
 
  int old_errno = errno;
287
 
  int ret = 0;
288
 
  if(seteuid(0) == -1){
289
 
    ret = errno;
290
 
  }
291
 
  errno = old_errno;
292
 
  return ret;
293
 
}
294
 
 
295
 
/* Set effective and real user ID to 0.  Return errno. */
296
 
__attribute__((warn_unused_result))
297
 
int raise_privileges_permanently(void){
298
 
  int old_errno = errno;
299
 
  int ret = raise_privileges();
300
 
  if(ret != 0){
301
 
    errno = old_errno;
302
 
    return ret;
303
 
  }
304
 
  if(setuid(0) == -1){
305
 
    ret = errno;
306
 
  }
307
 
  errno = old_errno;
308
 
  return ret;
309
 
}
310
 
 
311
 
/* Set effective user ID to unprivileged saved user ID */
312
 
__attribute__((warn_unused_result))
313
 
int lower_privileges(void){
314
 
  int old_errno = errno;
315
 
  int ret = 0;
316
 
  if(seteuid(uid) == -1){
317
 
    ret = errno;
318
 
  }
319
 
  errno = old_errno;
320
 
  return ret;
321
 
}
322
 
 
323
 
/* Lower privileges permanently */
324
 
__attribute__((warn_unused_result))
325
 
int lower_privileges_permanently(void){
326
 
  int old_errno = errno;
327
 
  int ret = 0;
328
 
  if(setuid(uid) == -1){
329
 
    ret = errno;
330
 
  }
331
 
  errno = old_errno;
332
 
  return ret;
333
 
}
334
 
 
335
275
/* 
336
276
 * Initialize GPGME.
337
277
 */
357
297
      return false;
358
298
    }
359
299
    
360
 
    /* Workaround for systems without a real-time clock; see also
361
 
       Debian bug #894495: <https://bugs.debian.org/894495> */
362
 
    do {
363
 
      {
364
 
        time_t currtime = time(NULL);
365
 
        if(currtime != (time_t)-1){
366
 
          struct tm tm;
367
 
          if(gmtime_r(&currtime, &tm) == NULL) {
368
 
            perror_plus("gmtime_r");
369
 
            break;
370
 
          }
371
 
          if(tm.tm_year != 70 or tm.tm_mon != 0){
372
 
            break;
373
 
          }
374
 
          if(debug){
375
 
            fprintf_plus(stderr, "System clock is January 1970");
376
 
          }
377
 
        } else {
378
 
          if(debug){
379
 
            fprintf_plus(stderr, "System clock is invalid");
380
 
          }
381
 
        }
382
 
      }
383
 
      struct stat keystat;
384
 
      ret = fstat(fd, &keystat);
385
 
      if(ret != 0){
386
 
        perror_plus("fstat");
387
 
        break;
388
 
      }
389
 
      ret = raise_privileges();
390
 
      if(ret != 0){
391
 
        errno = ret;
392
 
        perror_plus("Failed to raise privileges");
393
 
        break;
394
 
      }
395
 
      if(debug){
396
 
        fprintf_plus(stderr,
397
 
                     "Setting system clock to key file mtime");
398
 
      }
399
 
      time_t keytime = keystat.st_mtim.tv_sec;
400
 
      if(stime(&keytime) != 0){
401
 
        perror_plus("stime");
402
 
      }
403
 
      ret = lower_privileges();
404
 
      if(ret != 0){
405
 
        errno = ret;
406
 
        perror_plus("Failed to lower privileges");
407
 
      }
408
 
    } while(false);
409
 
 
410
300
    rc = gpgme_data_new_from_fd(&pgp_data, fd);
411
301
    if(rc != GPG_ERR_NO_ERROR){
412
302
      fprintf_plus(stderr, "bad gpgme_data_new_from_fd: %s: %s\n",
420
310
                   gpgme_strsource(rc), gpgme_strerror(rc));
421
311
      return false;
422
312
    }
423
 
    {
424
 
      gpgme_import_result_t import_result
425
 
        = gpgme_op_import_result(mc->ctx);
426
 
      if((import_result->imported < 1
427
 
          or import_result->not_imported > 0)
428
 
         and import_result->unchanged == 0){
429
 
        fprintf_plus(stderr, "bad gpgme_op_import_results:\n");
430
 
        fprintf_plus(stderr,
431
 
                     "The total number of considered keys: %d\n",
432
 
                     import_result->considered);
433
 
        fprintf_plus(stderr,
434
 
                     "The number of keys without user ID: %d\n",
435
 
                     import_result->no_user_id);
436
 
        fprintf_plus(stderr,
437
 
                     "The total number of imported keys: %d\n",
438
 
                     import_result->imported);
439
 
        fprintf_plus(stderr, "The number of imported RSA keys: %d\n",
440
 
                     import_result->imported_rsa);
441
 
        fprintf_plus(stderr, "The number of unchanged keys: %d\n",
442
 
                     import_result->unchanged);
443
 
        fprintf_plus(stderr, "The number of new user IDs: %d\n",
444
 
                     import_result->new_user_ids);
445
 
        fprintf_plus(stderr, "The number of new sub keys: %d\n",
446
 
                     import_result->new_sub_keys);
447
 
        fprintf_plus(stderr, "The number of new signatures: %d\n",
448
 
                     import_result->new_signatures);
449
 
        fprintf_plus(stderr, "The number of new revocations: %d\n",
450
 
                     import_result->new_revocations);
451
 
        fprintf_plus(stderr,
452
 
                     "The total number of secret keys read: %d\n",
453
 
                     import_result->secret_read);
454
 
        fprintf_plus(stderr,
455
 
                     "The number of imported secret keys: %d\n",
456
 
                     import_result->secret_imported);
457
 
        fprintf_plus(stderr,
458
 
                     "The number of unchanged secret keys: %d\n",
459
 
                     import_result->secret_unchanged);
460
 
        fprintf_plus(stderr, "The number of keys not imported: %d\n",
461
 
                     import_result->not_imported);
462
 
        for(gpgme_import_status_t import_status
463
 
              = import_result->imports;
464
 
            import_status != NULL;
465
 
            import_status = import_status->next){
466
 
          fprintf_plus(stderr, "Import status for key: %s\n",
467
 
                       import_status->fpr);
468
 
          if(import_status->result != GPG_ERR_NO_ERROR){
469
 
            fprintf_plus(stderr, "Import result: %s: %s\n",
470
 
                         gpgme_strsource(import_status->result),
471
 
                         gpgme_strerror(import_status->result));
472
 
          }
473
 
          fprintf_plus(stderr, "Key status:\n");
474
 
          fprintf_plus(stderr,
475
 
                       import_status->status & GPGME_IMPORT_NEW
476
 
                       ? "The key was new.\n"
477
 
                       : "The key was not new.\n");
478
 
          fprintf_plus(stderr,
479
 
                       import_status->status & GPGME_IMPORT_UID
480
 
                       ? "The key contained new user IDs.\n"
481
 
                       : "The key did not contain new user IDs.\n");
482
 
          fprintf_plus(stderr,
483
 
                       import_status->status & GPGME_IMPORT_SIG
484
 
                       ? "The key contained new signatures.\n"
485
 
                       : "The key did not contain new signatures.\n");
486
 
          fprintf_plus(stderr,
487
 
                       import_status->status & GPGME_IMPORT_SUBKEY
488
 
                       ? "The key contained new sub keys.\n"
489
 
                       : "The key did not contain new sub keys.\n");
490
 
          fprintf_plus(stderr,
491
 
                       import_status->status & GPGME_IMPORT_SECRET
492
 
                       ? "The key contained a secret key.\n"
493
 
                       : "The key did not contain a secret key.\n");
494
 
        }
495
 
        return false;
496
 
      }
497
 
    }
498
313
    
499
314
    ret = close(fd);
500
315
    if(ret == -1){
541
356
  /* Create new GPGME "context" */
542
357
  rc = gpgme_new(&(mc->ctx));
543
358
  if(rc != GPG_ERR_NO_ERROR){
544
 
    fprintf_plus(stderr, "bad gpgme_new: %s: %s\n",
545
 
                 gpgme_strsource(rc), gpgme_strerror(rc));
 
359
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
360
                 "bad gpgme_new: %s: %s\n", gpgme_strsource(rc),
 
361
                 gpgme_strerror(rc));
546
362
    return false;
547
363
  }
548
364
  
584
400
  /* Create new empty GPGME data buffer for the plaintext */
585
401
  rc = gpgme_data_new(&dh_plain);
586
402
  if(rc != GPG_ERR_NO_ERROR){
587
 
    fprintf_plus(stderr, "bad gpgme_data_new: %s: %s\n",
 
403
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
404
                 "bad gpgme_data_new: %s: %s\n",
588
405
                 gpgme_strsource(rc), gpgme_strerror(rc));
589
406
    gpgme_data_release(dh_crypto);
590
407
    return -1;
603
420
      if(result == NULL){
604
421
        fprintf_plus(stderr, "gpgme_op_decrypt_result failed\n");
605
422
      } else {
606
 
        if(result->unsupported_algorithm != NULL) {
607
 
          fprintf_plus(stderr, "Unsupported algorithm: %s\n",
608
 
                       result->unsupported_algorithm);
609
 
        }
610
 
        fprintf_plus(stderr, "Wrong key usage: %s\n",
611
 
                     result->wrong_key_usage ? "Yes" : "No");
 
423
        fprintf_plus(stderr, "Unsupported algorithm: %s\n",
 
424
                     result->unsupported_algorithm);
 
425
        fprintf_plus(stderr, "Wrong key usage: %u\n",
 
426
                     result->wrong_key_usage);
612
427
        if(result->file_name != NULL){
613
428
          fprintf_plus(stderr, "File name: %s\n", result->file_name);
614
429
        }
615
 
 
616
 
        for(gpgme_recipient_t r = result->recipients; r != NULL;
617
 
            r = r->next){
 
430
        gpgme_recipient_t recipient;
 
431
        recipient = result->recipients;
 
432
        while(recipient != NULL){
618
433
          fprintf_plus(stderr, "Public key algorithm: %s\n",
619
 
                       gpgme_pubkey_algo_name(r->pubkey_algo));
620
 
          fprintf_plus(stderr, "Key ID: %s\n", r->keyid);
 
434
                       gpgme_pubkey_algo_name
 
435
                       (recipient->pubkey_algo));
 
436
          fprintf_plus(stderr, "Key ID: %s\n", recipient->keyid);
621
437
          fprintf_plus(stderr, "Secret key available: %s\n",
622
 
                       r->status == GPG_ERR_NO_SECKEY ? "No" : "Yes");
 
438
                       recipient->status == GPG_ERR_NO_SECKEY
 
439
                       ? "No" : "Yes");
 
440
          recipient = recipient->next;
623
441
        }
624
442
      }
625
443
    }
707
525
                              const char *dhparamsfilename,
708
526
                              mandos_context *mc){
709
527
  int ret;
 
528
  unsigned int uret;
710
529
  
711
530
  if(debug){
712
531
    fprintf_plus(stderr, "Initializing GnuTLS\n");
729
548
  }
730
549
  
731
550
  if(debug){
732
 
    fprintf_plus(stderr, "Attempting to use public key %s and"
733
 
                 " private key %s as GnuTLS credentials\n",
 
551
    fprintf_plus(stderr, "Attempting to use OpenPGP public key %s and"
 
552
                 " secret key %s as GnuTLS credentials\n",
734
553
                 pubkeyfilename,
735
554
                 seckeyfilename);
736
555
  }
737
556
  
738
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
739
 
  ret = gnutls_certificate_set_rawpk_key_file
740
 
    (mc->cred, pubkeyfilename, seckeyfilename,
741
 
     GNUTLS_X509_FMT_PEM,       /* format */
742
 
     NULL,                      /* pass */
743
 
     /* key_usage */
744
 
     GNUTLS_KEY_DIGITAL_SIGNATURE | GNUTLS_KEY_KEY_ENCIPHERMENT,
745
 
     NULL,                      /* names */
746
 
     0,                         /* names_length */
747
 
     /* privkey_flags */
748
 
     GNUTLS_PKCS_PLAIN | GNUTLS_PKCS_NULL_PASSWORD,
749
 
     0);                        /* pkcs11_flags */
750
 
#elif GNUTLS_VERSION_NUMBER < 0x030600
751
557
  ret = gnutls_certificate_set_openpgp_key_file
752
558
    (mc->cred, pubkeyfilename, seckeyfilename,
753
559
     GNUTLS_OPENPGP_FMT_BASE64);
754
 
#else
755
 
#error "Needs GnuTLS 3.6.6 or later, or before 3.6.0"
756
 
#endif
757
560
  if(ret != GNUTLS_E_SUCCESS){
758
561
    fprintf_plus(stderr,
759
 
                 "Error[%d] while reading the key pair ('%s',"
 
562
                 "Error[%d] while reading the OpenPGP key pair ('%s',"
760
563
                 " '%s')\n", ret, pubkeyfilename, seckeyfilename);
761
564
    fprintf_plus(stderr, "The GnuTLS error is: %s\n",
762
565
                 safer_gnutls_strerror(ret));
810
613
        }
811
614
        params.size += (unsigned int)bytes_read;
812
615
      }
813
 
      ret = close(dhpfile);
814
 
      if(ret == -1){
815
 
        perror_plus("close");
816
 
      }
817
616
      if(params.data == NULL){
818
617
        dhparamsfilename = NULL;
819
618
      }
833
632
  }
834
633
  if(dhparamsfilename == NULL){
835
634
    if(mc->dh_bits == 0){
836
 
#if GNUTLS_VERSION_NUMBER < 0x030600
837
635
      /* Find out the optimal number of DH bits */
838
636
      /* Try to read the private key file */
839
637
      gnutls_datum_t buffer = { .data = NULL, .size = 0 };
919
717
          }
920
718
        }
921
719
      }
922
 
      unsigned int uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
 
720
      uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
923
721
      if(uret != 0){
924
722
        mc->dh_bits = uret;
925
723
        if(debug){
937
735
                     safer_gnutls_strerror(ret));
938
736
        goto globalfail;
939
737
      }
940
 
#endif
941
 
    } else {                    /* dh_bits != 0 */
942
 
      if(debug){
943
 
        fprintf_plus(stderr, "DH bits explicitly set to %u\n",
944
 
                     mc->dh_bits);
945
 
      }
946
 
      ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
947
 
      if(ret != GNUTLS_E_SUCCESS){
948
 
        fprintf_plus(stderr, "Error in GnuTLS prime generation (%u"
949
 
                     " bits): %s\n", mc->dh_bits,
950
 
                     safer_gnutls_strerror(ret));
951
 
        goto globalfail;
952
 
      }
953
 
      gnutls_certificate_set_dh_params(mc->cred, mc->dh_params);
 
738
    } else if(debug){
 
739
      fprintf_plus(stderr, "DH bits explicitly set to %u\n",
 
740
                   mc->dh_bits);
 
741
    }
 
742
    ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
 
743
    if(ret != GNUTLS_E_SUCCESS){
 
744
      fprintf_plus(stderr, "Error in GnuTLS prime generation (%u"
 
745
                   " bits): %s\n", mc->dh_bits,
 
746
                   safer_gnutls_strerror(ret));
 
747
      goto globalfail;
954
748
    }
955
749
  }
 
750
  gnutls_certificate_set_dh_params(mc->cred, mc->dh_params);
956
751
  
957
752
  return 0;
958
753
  
969
764
  int ret;
970
765
  /* GnuTLS session creation */
971
766
  do {
972
 
    ret = gnutls_init(session, (GNUTLS_SERVER
973
 
#if GNUTLS_VERSION_NUMBER >= 0x030506
974
 
                                | GNUTLS_NO_TICKETS
975
 
#endif
976
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
977
 
                                | GNUTLS_ENABLE_RAWPK
978
 
#endif
979
 
                                ));
 
767
    ret = gnutls_init(session, GNUTLS_SERVER);
980
768
    if(quit_now){
981
769
      return -1;
982
770
    }
1030
818
static void empty_log(__attribute__((unused)) AvahiLogLevel level,
1031
819
                      __attribute__((unused)) const char *txt){}
1032
820
 
 
821
/* Set effective uid to 0, return errno */
 
822
__attribute__((warn_unused_result))
 
823
int raise_privileges(void){
 
824
  int old_errno = errno;
 
825
  int ret = 0;
 
826
  if(seteuid(0) == -1){
 
827
    ret = errno;
 
828
  }
 
829
  errno = old_errno;
 
830
  return ret;
 
831
}
 
832
 
 
833
/* Set effective and real user ID to 0.  Return errno. */
 
834
__attribute__((warn_unused_result))
 
835
int raise_privileges_permanently(void){
 
836
  int old_errno = errno;
 
837
  int ret = raise_privileges();
 
838
  if(ret != 0){
 
839
    errno = old_errno;
 
840
    return ret;
 
841
  }
 
842
  if(setuid(0) == -1){
 
843
    ret = errno;
 
844
  }
 
845
  errno = old_errno;
 
846
  return ret;
 
847
}
 
848
 
 
849
/* Set effective user ID to unprivileged saved user ID */
 
850
__attribute__((warn_unused_result))
 
851
int lower_privileges(void){
 
852
  int old_errno = errno;
 
853
  int ret = 0;
 
854
  if(seteuid(uid) == -1){
 
855
    ret = errno;
 
856
  }
 
857
  errno = old_errno;
 
858
  return ret;
 
859
}
 
860
 
 
861
/* Lower privileges permanently */
 
862
__attribute__((warn_unused_result))
 
863
int lower_privileges_permanently(void){
 
864
  int old_errno = errno;
 
865
  int ret = 0;
 
866
  if(setuid(uid) == -1){
 
867
    ret = errno;
 
868
  }
 
869
  errno = old_errno;
 
870
  return ret;
 
871
}
 
872
 
1033
873
/* Helper function to add_local_route() and delete_local_route() */
1034
874
__attribute__((nonnull, warn_unused_result))
1035
875
static bool add_delete_local_route(const bool add,
1815
1655
      perror_plus("ioctl SIOCGIFFLAGS");
1816
1656
      errno = old_errno;
1817
1657
    }
1818
 
    if((close(s) == -1) and debug){
1819
 
      old_errno = errno;
1820
 
      perror_plus("close");
1821
 
      errno = old_errno;
1822
 
    }
1823
1658
    return false;
1824
1659
  }
1825
 
  if((close(s) == -1) and debug){
1826
 
    old_errno = errno;
1827
 
    perror_plus("close");
1828
 
    errno = old_errno;
1829
 
  }
1830
1660
  return true;
1831
1661
}
1832
1662
 
2461
2291
 
2462
2292
int main(int argc, char *argv[]){
2463
2293
  mandos_context mc = { .server = NULL, .dh_bits = 0,
2464
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2465
 
                        .priority = "SECURE128:!CTYPE-X.509"
2466
 
                        ":+CTYPE-RAWPK:!RSA:!VERS-ALL:+VERS-TLS1.3"
2467
 
                        ":%PROFILE_ULTRA",
2468
 
#elif GNUTLS_VERSION_NUMBER < 0x030600
2469
2294
                        .priority = "SECURE256:!CTYPE-X.509"
2470
2295
                        ":+CTYPE-OPENPGP:!RSA:+SIGN-DSA-SHA256",
2471
 
#else
2472
 
#error "Needs GnuTLS 3.6.6 or later, or before 3.6.0"
2473
 
#endif
2474
2296
                        .current_server = NULL, .interfaces = NULL,
2475
2297
                        .interfaces_size = 0 };
2476
2298
  AvahiSServiceBrowser *sb = NULL;
2487
2309
  AvahiIfIndex if_index = AVAHI_IF_UNSPEC;
2488
2310
  const char *seckey = PATHDIR "/" SECKEY;
2489
2311
  const char *pubkey = PATHDIR "/" PUBKEY;
2490
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2491
 
  const char *tls_privkey = PATHDIR "/" TLS_PRIVKEY;
2492
 
  const char *tls_pubkey = PATHDIR "/" TLS_PUBKEY;
2493
 
#endif
2494
2312
  const char *dh_params_file = NULL;
2495
2313
  char *interfaces_hooks = NULL;
2496
2314
  
2544
2362
      { .name = "pubkey", .key = 'p',
2545
2363
        .arg = "FILE",
2546
2364
        .doc = "OpenPGP public key file base name",
2547
 
        .group = 1 },
2548
 
      { .name = "tls-privkey", .key = 't',
2549
 
        .arg = "FILE",
2550
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2551
 
        .doc = "TLS private key file base name",
2552
 
#else
2553
 
        .doc = "Dummy; ignored (requires GnuTLS 3.6.6)",
2554
 
#endif
2555
 
        .group = 1 },
2556
 
      { .name = "tls-pubkey", .key = 'T',
2557
 
        .arg = "FILE",
2558
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2559
 
        .doc = "TLS public key file base name",
2560
 
#else
2561
 
        .doc = "Dummy; ignored (requires GnuTLS 3.6.6)",
2562
 
#endif
2563
 
        .group = 1 },
 
2365
        .group = 2 },
2564
2366
      { .name = "dh-bits", .key = 129,
2565
2367
        .arg = "BITS",
2566
2368
        .doc = "Bit length of the prime number used in the"
2622
2424
      case 'p':                 /* --pubkey */
2623
2425
        pubkey = arg;
2624
2426
        break;
2625
 
      case 't':                 /* --tls-privkey */
2626
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2627
 
        tls_privkey = arg;
2628
 
#endif
2629
 
        break;
2630
 
      case 'T':                 /* --tls-pubkey */
2631
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2632
 
        tls_pubkey = arg;
2633
 
#endif
2634
 
        break;
2635
2427
      case 129:                 /* --dh-bits */
2636
2428
        errno = 0;
2637
2429
        tmpmax = strtoimax(arg, &tmp, 10);
2991
2783
    goto end;
2992
2784
  }
2993
2785
  
2994
 
#if GNUTLS_VERSION_NUMBER >= 0x030606
2995
 
  ret = init_gnutls_global(tls_pubkey, tls_privkey, dh_params_file, &mc);
2996
 
#elif GNUTLS_VERSION_NUMBER < 0x030600
2997
2786
  ret = init_gnutls_global(pubkey, seckey, dh_params_file, &mc);
2998
 
#else
2999
 
#error "Needs GnuTLS 3.6.6 or later, or before 3.6.0"
3000
 
#endif
3001
2787
  if(ret == -1){
3002
2788
    fprintf_plus(stderr, "init_gnutls_global failed\n");
3003
2789
    exitcode = EX_UNAVAILABLE;
3276
3062
                                                | O_PATH));
3277
3063
    if(dir_fd == -1){
3278
3064
      perror_plus("open");
3279
 
      return;
3280
3065
    }
3281
3066
    int numentries = scandirat(dir_fd, ".", &direntries,
3282
3067
                               notdotentries, alphasort);
3309
3094
      
3310
3095
      /* need to clean even if 0 because man page doesn't specify */
3311
3096
      free(direntries);
 
3097
      if(numentries == -1){
 
3098
        perror_plus("scandirat");
 
3099
      }
3312
3100
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3313
3101
      if(dret == -1 and errno != ENOENT){
3314
3102
        perror_plus("rmdir");