/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: 2016-03-17 20:40:55 UTC
  • Revision ID: teddy@recompile.se-20160317204055-bhsh5xsidq7w5cxu
Client: Fix plymouth agent; broken since 1.7.2.

Fix an very old memory bug in the plymouth agent (which has been
present since its apperance in version 1.2), but which was only
recently detected at run time due to the new -fsanitize=address
compile- time flag, which has been used since version 1.7.2.  This
detection of a memory access violation causes the program to abort,
making the Plymouth graphical boot system unable to accept interactive
input of passwords when using the Mandos client.

* plugins.d/plymouth.c (exec_and_wait): Fix memory allocation bug when
  allocating new_argv.  Also tolerate a zero-length argv.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008-2019 Teddy Hogeborn
13
 
 * Copyright © 2008-2019 Björn Påhlsson
14
 
 * 
15
 
 * This file is part of Mandos.
16
 
 * 
17
 
 * Mandos is free software: you can redistribute it and/or modify it
18
 
 * under the terms of the GNU General Public License as published by
19
 
 * the Free Software Foundation, either version 3 of the License, or
20
 
 * (at your option) any later version.
21
 
 * 
22
 
 * Mandos is distributed in the hope that it will be useful, but
 
12
 * Copyright © 2008-2016 Teddy Hogeborn
 
13
 * Copyright © 2008-2016 Björn Påhlsson
 
14
 * 
 
15
 * This program is free software: you can redistribute it and/or
 
16
 * modify it under the terms of the GNU General Public License as
 
17
 * published by the Free Software Foundation, either version 3 of the
 
18
 * License, or (at your option) any later version.
 
19
 * 
 
20
 * This program is distributed in the hope that it will be useful, but
23
21
 * WITHOUT ANY WARRANTY; without even the implied warranty of
24
22
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
25
23
 * General Public License for more details.
26
24
 * 
27
25
 * You should have received a copy of the GNU General Public License
28
 
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
 
26
 * along with this program.  If not, see
 
27
 * <http://www.gnu.org/licenses/>.
29
28
 * 
30
29
 * Contact the authors at <mandos@recompile.se>.
31
30
 */
48
47
                                   strtof(), abort() */
49
48
#include <stdbool.h>            /* bool, false, true */
50
49
#include <string.h>             /* strcmp(), strlen(), strerror(),
51
 
                                   asprintf(), strncpy(), strsignal()
52
 
                                */
 
50
                                   asprintf(), strncpy() */
53
51
#include <sys/ioctl.h>          /* ioctl */
54
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
55
53
                                   sockaddr_in6, PF_INET6,
123
121
                                   gnutls_*
124
122
                                   init_gnutls_session(),
125
123
                                   GNUTLS_* */
126
 
#if GNUTLS_VERSION_NUMBER < 0x030600
127
124
#include <gnutls/openpgp.h>
128
125
                         /* gnutls_certificate_set_openpgp_key_file(),
129
126
                            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
127
 
136
128
/* GPGME */
137
129
#include <gpgme.h>              /* All GPGME types, constants and
145
137
#define PATHDIR "/conf/conf.d/mandos"
146
138
#define SECKEY "seckey.txt"
147
139
#define PUBKEY "pubkey.txt"
148
 
#define TLS_PRIVKEY "tls-privkey.pem"
149
 
#define TLS_PUBKEY "tls-pubkey.pem"
150
140
#define HOOKDIR "/lib/mandos/network-hooks.d"
151
141
 
152
142
bool debug = false;
280
270
  return true;
281
271
}
282
272
 
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
273
/* 
336
274
 * Initialize GPGME.
337
275
 */
357
295
      return false;
358
296
    }
359
297
    
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
298
    rc = gpgme_data_new_from_fd(&pgp_data, fd);
411
299
    if(rc != GPG_ERR_NO_ERROR){
412
300
      fprintf_plus(stderr, "bad gpgme_data_new_from_fd: %s: %s\n",
420
308
                   gpgme_strsource(rc), gpgme_strerror(rc));
421
309
      return false;
422
310
    }
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
311
    
499
312
    ret = close(fd);
500
313
    if(ret == -1){
541
354
  /* Create new GPGME "context" */
542
355
  rc = gpgme_new(&(mc->ctx));
543
356
  if(rc != GPG_ERR_NO_ERROR){
544
 
    fprintf_plus(stderr, "bad gpgme_new: %s: %s\n",
545
 
                 gpgme_strsource(rc), gpgme_strerror(rc));
 
357
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
358
                 "bad gpgme_new: %s: %s\n", gpgme_strsource(rc),
 
359
                 gpgme_strerror(rc));
546
360
    return false;
547
361
  }
548
362
  
584
398
  /* Create new empty GPGME data buffer for the plaintext */
585
399
  rc = gpgme_data_new(&dh_plain);
586
400
  if(rc != GPG_ERR_NO_ERROR){
587
 
    fprintf_plus(stderr, "bad gpgme_data_new: %s: %s\n",
 
401
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
402
                 "bad gpgme_data_new: %s: %s\n",
588
403
                 gpgme_strsource(rc), gpgme_strerror(rc));
589
404
    gpgme_data_release(dh_crypto);
590
405
    return -1;
603
418
      if(result == NULL){
604
419
        fprintf_plus(stderr, "gpgme_op_decrypt_result failed\n");
605
420
      } 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");
 
421
        fprintf_plus(stderr, "Unsupported algorithm: %s\n",
 
422
                     result->unsupported_algorithm);
 
423
        fprintf_plus(stderr, "Wrong key usage: %u\n",
 
424
                     result->wrong_key_usage);
612
425
        if(result->file_name != NULL){
613
426
          fprintf_plus(stderr, "File name: %s\n", result->file_name);
614
427
        }
615
 
 
616
 
        for(gpgme_recipient_t r = result->recipients; r != NULL;
617
 
            r = r->next){
 
428
        gpgme_recipient_t recipient;
 
429
        recipient = result->recipients;
 
430
        while(recipient != NULL){
618
431
          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);
 
432
                       gpgme_pubkey_algo_name
 
433
                       (recipient->pubkey_algo));
 
434
          fprintf_plus(stderr, "Key ID: %s\n", recipient->keyid);
621
435
          fprintf_plus(stderr, "Secret key available: %s\n",
622
 
                       r->status == GPG_ERR_NO_SECKEY ? "No" : "Yes");
 
436
                       recipient->status == GPG_ERR_NO_SECKEY
 
437
                       ? "No" : "Yes");
 
438
          recipient = recipient->next;
623
439
        }
624
440
      }
625
441
    }
707
523
                              const char *dhparamsfilename,
708
524
                              mandos_context *mc){
709
525
  int ret;
 
526
  unsigned int uret;
710
527
  
711
528
  if(debug){
712
529
    fprintf_plus(stderr, "Initializing GnuTLS\n");
729
546
  }
730
547
  
731
548
  if(debug){
732
 
    fprintf_plus(stderr, "Attempting to use public key %s and"
733
 
                 " private key %s as GnuTLS credentials\n",
 
549
    fprintf_plus(stderr, "Attempting to use OpenPGP public key %s and"
 
550
                 " secret key %s as GnuTLS credentials\n",
734
551
                 pubkeyfilename,
735
552
                 seckeyfilename);
736
553
  }
737
554
  
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
555
  ret = gnutls_certificate_set_openpgp_key_file
752
556
    (mc->cred, pubkeyfilename, seckeyfilename,
753
557
     GNUTLS_OPENPGP_FMT_BASE64);
754
 
#else
755
 
#error "Needs GnuTLS 3.6.6 or later, or before 3.6.0"
756
 
#endif
757
558
  if(ret != GNUTLS_E_SUCCESS){
758
559
    fprintf_plus(stderr,
759
 
                 "Error[%d] while reading the key pair ('%s',"
 
560
                 "Error[%d] while reading the OpenPGP key pair ('%s',"
760
561
                 " '%s')\n", ret, pubkeyfilename, seckeyfilename);
761
562
    fprintf_plus(stderr, "The GnuTLS error is: %s\n",
762
563
                 safer_gnutls_strerror(ret));
810
611
        }
811
612
        params.size += (unsigned int)bytes_read;
812
613
      }
813
 
      ret = close(dhpfile);
814
 
      if(ret == -1){
815
 
        perror_plus("close");
816
 
      }
817
614
      if(params.data == NULL){
818
615
        dhparamsfilename = NULL;
819
616
      }
828
625
                     safer_gnutls_strerror(ret));
829
626
        dhparamsfilename = NULL;
830
627
      }
831
 
      free(params.data);
832
628
    } while(false);
833
629
  }
834
630
  if(dhparamsfilename == NULL){
835
631
    if(mc->dh_bits == 0){
836
 
#if GNUTLS_VERSION_NUMBER < 0x030600
837
632
      /* Find out the optimal number of DH bits */
838
633
      /* Try to read the private key file */
839
634
      gnutls_datum_t buffer = { .data = NULL, .size = 0 };
919
714
          }
920
715
        }
921
716
      }
922
 
      unsigned int uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
 
717
      uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
923
718
      if(uret != 0){
924
719
        mc->dh_bits = uret;
925
720
        if(debug){
937
732
                     safer_gnutls_strerror(ret));
938
733
        goto globalfail;
939
734
      }
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);
 
735
    } else if(debug){
 
736
      fprintf_plus(stderr, "DH bits explicitly set to %u\n",
 
737
                   mc->dh_bits);
 
738
    }
 
739
    ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
 
740
    if(ret != GNUTLS_E_SUCCESS){
 
741
      fprintf_plus(stderr, "Error in GnuTLS prime generation (%u"
 
742
                   " bits): %s\n", mc->dh_bits,
 
743
                   safer_gnutls_strerror(ret));
 
744
      goto globalfail;
954
745
    }
955
746
  }
 
747
  gnutls_certificate_set_dh_params(mc->cred, mc->dh_params);
956
748
  
957
749
  return 0;
958
750
  
969
761
  int ret;
970
762
  /* GnuTLS session creation */
971
763
  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
 
                                ));
 
764
    ret = gnutls_init(session, GNUTLS_SERVER);
980
765
    if(quit_now){
981
766
      return -1;
982
767
    }
1030
815
static void empty_log(__attribute__((unused)) AvahiLogLevel level,
1031
816
                      __attribute__((unused)) const char *txt){}
1032
817
 
 
818
/* Set effective uid to 0, return errno */
 
819
__attribute__((warn_unused_result))
 
820
int raise_privileges(void){
 
821
  int old_errno = errno;
 
822
  int ret = 0;
 
823
  if(seteuid(0) == -1){
 
824
    ret = errno;
 
825
  }
 
826
  errno = old_errno;
 
827
  return ret;
 
828
}
 
829
 
 
830
/* Set effective and real user ID to 0.  Return errno. */
 
831
__attribute__((warn_unused_result))
 
832
int raise_privileges_permanently(void){
 
833
  int old_errno = errno;
 
834
  int ret = raise_privileges();
 
835
  if(ret != 0){
 
836
    errno = old_errno;
 
837
    return ret;
 
838
  }
 
839
  if(setuid(0) == -1){
 
840
    ret = errno;
 
841
  }
 
842
  errno = old_errno;
 
843
  return ret;
 
844
}
 
845
 
 
846
/* Set effective user ID to unprivileged saved user ID */
 
847
__attribute__((warn_unused_result))
 
848
int lower_privileges(void){
 
849
  int old_errno = errno;
 
850
  int ret = 0;
 
851
  if(seteuid(uid) == -1){
 
852
    ret = errno;
 
853
  }
 
854
  errno = old_errno;
 
855
  return ret;
 
856
}
 
857
 
 
858
/* Lower privileges permanently */
 
859
__attribute__((warn_unused_result))
 
860
int lower_privileges_permanently(void){
 
861
  int old_errno = errno;
 
862
  int ret = 0;
 
863
  if(setuid(uid) == -1){
 
864
    ret = errno;
 
865
  }
 
866
  errno = old_errno;
 
867
  return ret;
 
868
}
 
869
 
1033
870
/* Helper function to add_local_route() and delete_local_route() */
1034
871
__attribute__((nonnull, warn_unused_result))
1035
872
static bool add_delete_local_route(const bool add,
1240
1077
    bool match = false;
1241
1078
    {
1242
1079
      char *interface = NULL;
1243
 
      while((interface = argz_next(mc->interfaces,
1244
 
                                   mc->interfaces_size,
1245
 
                                   interface))){
 
1080
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
 
1081
                                 interface))){
1246
1082
        if(if_nametoindex(interface) == (unsigned int)if_index){
1247
1083
          match = true;
1248
1084
          break;
1401
1237
           with an explicit route added with the server's address.
1402
1238
           
1403
1239
           Avahi bug reference:
1404
 
           https://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1240
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
1405
1241
           https://bugs.debian.org/587961
1406
1242
        */
1407
1243
        if(debug){
1587
1423
                                               &decrypted_buffer, mc);
1588
1424
    if(decrypted_buffer_size >= 0){
1589
1425
      
1590
 
      clearerr(stdout);
1591
1426
      written = 0;
1592
1427
      while(written < (size_t) decrypted_buffer_size){
1593
1428
        if(quit_now){
1609
1444
        }
1610
1445
        written += (size_t)ret;
1611
1446
      }
1612
 
      ret = fflush(stdout);
1613
 
      if(ret != 0){
1614
 
        int e = errno;
1615
 
        if(debug){
1616
 
          fprintf_plus(stderr, "Error writing encrypted data: %s\n",
1617
 
                       strerror(errno));
1618
 
        }
1619
 
        errno = e;
1620
 
        goto mandos_end;
1621
 
      }
1622
1447
      retval = 0;
1623
1448
    }
1624
1449
  }
1655
1480
  return retval;
1656
1481
}
1657
1482
 
 
1483
__attribute__((nonnull))
1658
1484
static void resolve_callback(AvahiSServiceResolver *r,
1659
1485
                             AvahiIfIndex interface,
1660
1486
                             AvahiProtocol proto,
1815
1641
      perror_plus("ioctl SIOCGIFFLAGS");
1816
1642
      errno = old_errno;
1817
1643
    }
1818
 
    if((close(s) == -1) and debug){
1819
 
      old_errno = errno;
1820
 
      perror_plus("close");
1821
 
      errno = old_errno;
1822
 
    }
1823
1644
    return false;
1824
1645
  }
1825
 
  if((close(s) == -1) and debug){
1826
 
    old_errno = errno;
1827
 
    perror_plus("close");
1828
 
    errno = old_errno;
1829
 
  }
1830
1646
  return true;
1831
1647
}
1832
1648
 
2093
1909
      return;
2094
1910
    }
2095
1911
  }
2096
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
2097
 
  if(devnull == -1){
2098
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
2099
 
    return;
2100
 
  }
2101
1912
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
2102
1913
                           runnable_hook, alphasort);
2103
1914
  if(numhooks == -1){
2104
1915
    perror_plus("scandir");
2105
 
    close(devnull);
2106
1916
    return;
2107
1917
  }
2108
1918
  struct dirent *direntry;
2109
1919
  int ret;
 
1920
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1921
  if(devnull == -1){
 
1922
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1923
    return;
 
1924
  }
2110
1925
  for(int i = 0; i < numhooks; i++){
2111
1926
    direntry = direntries[i];
2112
1927
    if(debug){
2368
2183
  
2369
2184
  /* Sleep checking until interface is running.
2370
2185
     Check every 0.25s, up to total time of delay */
2371
 
  for(int i = 0; i < delay * 4; i++){
 
2186
  for(int i=0; i < delay * 4; i++){
2372
2187
    if(interface_is_running(interface)){
2373
2188
      break;
2374
2189
    }
2461
2276
 
2462
2277
int main(int argc, char *argv[]){
2463
2278
  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
2279
                        .priority = "SECURE256:!CTYPE-X.509"
2470
2280
                        ":+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
2281
                        .current_server = NULL, .interfaces = NULL,
2475
2282
                        .interfaces_size = 0 };
2476
2283
  AvahiSServiceBrowser *sb = NULL;
2487
2294
  AvahiIfIndex if_index = AVAHI_IF_UNSPEC;
2488
2295
  const char *seckey = PATHDIR "/" SECKEY;
2489
2296
  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
2297
  const char *dh_params_file = NULL;
2495
2298
  char *interfaces_hooks = NULL;
2496
2299
  
2544
2347
      { .name = "pubkey", .key = 'p',
2545
2348
        .arg = "FILE",
2546
2349
        .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 },
 
2350
        .group = 2 },
2564
2351
      { .name = "dh-bits", .key = 129,
2565
2352
        .arg = "BITS",
2566
2353
        .doc = "Bit length of the prime number used in the"
2622
2409
      case 'p':                 /* --pubkey */
2623
2410
        pubkey = arg;
2624
2411
        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
2412
      case 129:                 /* --dh-bits */
2636
2413
        errno = 0;
2637
2414
        tmpmax = strtoimax(arg, &tmp, 10);
2708
2485
  
2709
2486
  {
2710
2487
    /* Work around Debian bug #633582:
2711
 
       <https://bugs.debian.org/633582> */
 
2488
       <http://bugs.debian.org/633582> */
2712
2489
    
2713
2490
    /* Re-raise privileges */
2714
2491
    ret = raise_privileges();
2991
2768
    goto end;
2992
2769
  }
2993
2770
  
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
2771
  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
2772
  if(ret == -1){
3002
2773
    fprintf_plus(stderr, "init_gnutls_global failed\n");
3003
2774
    exitcode = EX_UNAVAILABLE;
3175
2946
 end:
3176
2947
  
3177
2948
  if(debug){
3178
 
    if(signal_received){
3179
 
      fprintf_plus(stderr, "%s exiting due to signal %d: %s\n",
3180
 
                   argv[0], signal_received,
3181
 
                   strsignal(signal_received));
3182
 
    } else {
3183
 
      fprintf_plus(stderr, "%s exiting\n", argv[0]);
3184
 
    }
 
2949
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
3185
2950
  }
3186
2951
  
3187
2952
  /* Cleanup things */
3239
3004
      /* Take down the network interfaces which were brought up */
3240
3005
      {
3241
3006
        char *interface = NULL;
3242
 
        while((interface = argz_next(interfaces_to_take_down,
3243
 
                                     interfaces_to_take_down_size,
3244
 
                                     interface))){
 
3007
        while((interface=argz_next(interfaces_to_take_down,
 
3008
                                   interfaces_to_take_down_size,
 
3009
                                   interface))){
3245
3010
          ret = take_down_interface(interface);
3246
3011
          if(ret != 0){
3247
3012
            errno = ret;
3276
3041
                                                | O_PATH));
3277
3042
    if(dir_fd == -1){
3278
3043
      perror_plus("open");
3279
 
      return;
3280
3044
    }
3281
3045
    int numentries = scandirat(dir_fd, ".", &direntries,
3282
3046
                               notdotentries, alphasort);
3299
3063
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3300
3064
            dret = 0;
3301
3065
          }
3302
 
          if((dret == -1) and (errno != ENOENT)){
 
3066
          if(dret == -1){
3303
3067
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3304
3068
                         direntries[i]->d_name, strerror(errno));
3305
3069
          }
3309
3073
      
3310
3074
      /* need to clean even if 0 because man page doesn't specify */
3311
3075
      free(direntries);
 
3076
      if(numentries == -1){
 
3077
        perror_plus("scandirat");
 
3078
      }
3312
3079
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3313
3080
      if(dret == -1 and errno != ENOENT){
3314
3081
        perror_plus("rmdir");