/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-04-01 19:27:30 UTC
  • Revision ID: teddy@recompile.se-20180401192730-jwr4io5cb3x5u268
Don't print NULL string pointers in error message output

* plugins.d/mandos-client.c (pgp_packet_decrypt): If decryption fails,
  don't print unsupported_algorithm if it is NULL.

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;
607
424
          fprintf_plus(stderr, "Unsupported algorithm: %s\n",
608
425
                       result->unsupported_algorithm);
609
426
        }
610
 
        fprintf_plus(stderr, "Wrong key usage: %s\n",
611
 
                     result->wrong_key_usage ? "Yes" : "No");
 
427
        fprintf_plus(stderr, "Wrong key usage: %u\n",
 
428
                     result->wrong_key_usage);
612
429
        if(result->file_name != NULL){
613
430
          fprintf_plus(stderr, "File name: %s\n", result->file_name);
614
431
        }
615
 
 
616
 
        for(gpgme_recipient_t r = result->recipients; r != NULL;
617
 
            r = r->next){
 
432
        gpgme_recipient_t recipient;
 
433
        recipient = result->recipients;
 
434
        while(recipient != NULL){
618
435
          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);
 
436
                       gpgme_pubkey_algo_name
 
437
                       (recipient->pubkey_algo));
 
438
          fprintf_plus(stderr, "Key ID: %s\n", recipient->keyid);
621
439
          fprintf_plus(stderr, "Secret key available: %s\n",
622
 
                       r->status == GPG_ERR_NO_SECKEY ? "No" : "Yes");
 
440
                       recipient->status == GPG_ERR_NO_SECKEY
 
441
                       ? "No" : "Yes");
 
442
          recipient = recipient->next;
623
443
        }
624
444
      }
625
445
    }
707
527
                              const char *dhparamsfilename,
708
528
                              mandos_context *mc){
709
529
  int ret;
 
530
  unsigned int uret;
710
531
  
711
532
  if(debug){
712
533
    fprintf_plus(stderr, "Initializing GnuTLS\n");
729
550
  }
730
551
  
731
552
  if(debug){
732
 
    fprintf_plus(stderr, "Attempting to use public key %s and"
733
 
                 " private key %s as GnuTLS credentials\n",
 
553
    fprintf_plus(stderr, "Attempting to use OpenPGP public key %s and"
 
554
                 " secret key %s as GnuTLS credentials\n",
734
555
                 pubkeyfilename,
735
556
                 seckeyfilename);
736
557
  }
737
558
  
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
559
  ret = gnutls_certificate_set_openpgp_key_file
752
560
    (mc->cred, pubkeyfilename, seckeyfilename,
753
561
     GNUTLS_OPENPGP_FMT_BASE64);
754
 
#else
755
 
#error "Needs GnuTLS 3.6.6 or later, or before 3.6.0"
756
 
#endif
757
562
  if(ret != GNUTLS_E_SUCCESS){
758
563
    fprintf_plus(stderr,
759
 
                 "Error[%d] while reading the key pair ('%s',"
 
564
                 "Error[%d] while reading the OpenPGP key pair ('%s',"
760
565
                 " '%s')\n", ret, pubkeyfilename, seckeyfilename);
761
566
    fprintf_plus(stderr, "The GnuTLS error is: %s\n",
762
567
                 safer_gnutls_strerror(ret));
833
638
  }
834
639
  if(dhparamsfilename == NULL){
835
640
    if(mc->dh_bits == 0){
836
 
#if GNUTLS_VERSION_NUMBER < 0x030600
837
641
      /* Find out the optimal number of DH bits */
838
642
      /* Try to read the private key file */
839
643
      gnutls_datum_t buffer = { .data = NULL, .size = 0 };
919
723
          }
920
724
        }
921
725
      }
922
 
      unsigned int uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
 
726
      uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
923
727
      if(uret != 0){
924
728
        mc->dh_bits = uret;
925
729
        if(debug){
937
741
                     safer_gnutls_strerror(ret));
938
742
        goto globalfail;
939
743
      }
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);
 
744
    } else if(debug){
 
745
      fprintf_plus(stderr, "DH bits explicitly set to %u\n",
 
746
                   mc->dh_bits);
 
747
    }
 
748
    ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
 
749
    if(ret != GNUTLS_E_SUCCESS){
 
750
      fprintf_plus(stderr, "Error in GnuTLS prime generation (%u"
 
751
                   " bits): %s\n", mc->dh_bits,
 
752
                   safer_gnutls_strerror(ret));
 
753
      goto globalfail;
954
754
    }
955
755
  }
 
756
  gnutls_certificate_set_dh_params(mc->cred, mc->dh_params);
956
757
  
957
758
  return 0;
958
759
  
969
770
  int ret;
970
771
  /* GnuTLS session creation */
971
772
  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
 
                                ));
 
773
    ret = gnutls_init(session, GNUTLS_SERVER);
980
774
    if(quit_now){
981
775
      return -1;
982
776
    }
1030
824
static void empty_log(__attribute__((unused)) AvahiLogLevel level,
1031
825
                      __attribute__((unused)) const char *txt){}
1032
826
 
 
827
/* Set effective uid to 0, return errno */
 
828
__attribute__((warn_unused_result))
 
829
int raise_privileges(void){
 
830
  int old_errno = errno;
 
831
  int ret = 0;
 
832
  if(seteuid(0) == -1){
 
833
    ret = errno;
 
834
  }
 
835
  errno = old_errno;
 
836
  return ret;
 
837
}
 
838
 
 
839
/* Set effective and real user ID to 0.  Return errno. */
 
840
__attribute__((warn_unused_result))
 
841
int raise_privileges_permanently(void){
 
842
  int old_errno = errno;
 
843
  int ret = raise_privileges();
 
844
  if(ret != 0){
 
845
    errno = old_errno;
 
846
    return ret;
 
847
  }
 
848
  if(setuid(0) == -1){
 
849
    ret = errno;
 
850
  }
 
851
  errno = old_errno;
 
852
  return ret;
 
853
}
 
854
 
 
855
/* Set effective user ID to unprivileged saved user ID */
 
856
__attribute__((warn_unused_result))
 
857
int lower_privileges(void){
 
858
  int old_errno = errno;
 
859
  int ret = 0;
 
860
  if(seteuid(uid) == -1){
 
861
    ret = errno;
 
862
  }
 
863
  errno = old_errno;
 
864
  return ret;
 
865
}
 
866
 
 
867
/* Lower privileges permanently */
 
868
__attribute__((warn_unused_result))
 
869
int lower_privileges_permanently(void){
 
870
  int old_errno = errno;
 
871
  int ret = 0;
 
872
  if(setuid(uid) == -1){
 
873
    ret = errno;
 
874
  }
 
875
  errno = old_errno;
 
876
  return ret;
 
877
}
 
878
 
1033
879
/* Helper function to add_local_route() and delete_local_route() */
1034
880
__attribute__((nonnull, warn_unused_result))
1035
881
static bool add_delete_local_route(const bool add,
2461
2307
 
2462
2308
int main(int argc, char *argv[]){
2463
2309
  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
2310
                        .priority = "SECURE256:!CTYPE-X.509"
2470
2311
                        ":+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
2312
                        .current_server = NULL, .interfaces = NULL,
2475
2313
                        .interfaces_size = 0 };
2476
2314
  AvahiSServiceBrowser *sb = NULL;
2487
2325
  AvahiIfIndex if_index = AVAHI_IF_UNSPEC;
2488
2326
  const char *seckey = PATHDIR "/" SECKEY;
2489
2327
  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
2328
  const char *dh_params_file = NULL;
2495
2329
  char *interfaces_hooks = NULL;
2496
2330
  
2544
2378
      { .name = "pubkey", .key = 'p',
2545
2379
        .arg = "FILE",
2546
2380
        .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 },
 
2381
        .group = 2 },
2564
2382
      { .name = "dh-bits", .key = 129,
2565
2383
        .arg = "BITS",
2566
2384
        .doc = "Bit length of the prime number used in the"
2622
2440
      case 'p':                 /* --pubkey */
2623
2441
        pubkey = arg;
2624
2442
        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
2443
      case 129:                 /* --dh-bits */
2636
2444
        errno = 0;
2637
2445
        tmpmax = strtoimax(arg, &tmp, 10);
2991
2799
    goto end;
2992
2800
  }
2993
2801
  
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
2802
  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
2803
  if(ret == -1){
3002
2804
    fprintf_plus(stderr, "init_gnutls_global failed\n");
3003
2805
    exitcode = EX_UNAVAILABLE;