/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to plugins.d/mandos-client.c

Merge change to add local route when network is "unreachable".

Show diffs side-by-side

added added

removed removed

Lines of Context:
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008-2014 Teddy Hogeborn
13
 
 * Copyright © 2008-2014 Björn Påhlsson
 
12
 * Copyright © 2008-2015 Teddy Hogeborn
 
13
 * Copyright © 2008-2015 Björn Påhlsson
14
14
 * 
15
15
 * This program is free software: you can redistribute it and/or
16
16
 * modify it under the terms of the GNU General Public License as
493
493
  return plaintext_length;
494
494
}
495
495
 
 
496
__attribute__((warn_unused_result, const))
 
497
static const char *safe_string(const char *str){
 
498
  if(str == NULL)
 
499
    return "(unknown)";
 
500
  return str;
 
501
}
 
502
 
496
503
__attribute__((warn_unused_result))
497
504
static const char *safer_gnutls_strerror(int value){
498
505
  const char *ret = gnutls_strerror(value);
499
 
  if(ret == NULL)
500
 
    ret = "(unknown)";
501
 
  return ret;
 
506
  return safe_string(ret);
502
507
}
503
508
 
504
509
/* GnuTLS log function callback */
513
518
                              const char *seckeyfilename,
514
519
                              mandos_context *mc){
515
520
  int ret;
 
521
  unsigned int uret;
516
522
  
517
523
  if(debug){
518
524
    fprintf_plus(stderr, "Initializing GnuTLS\n");
569
575
                 safer_gnutls_strerror(ret));
570
576
    goto globalfail;
571
577
  }
 
578
  if(mc->dh_bits == 0){
 
579
    /* Find out the optimal number of DH bits */
 
580
    /* Try to read the private key file */
 
581
    gnutls_datum_t buffer = { .data = NULL, .size = 0 };
 
582
    {
 
583
      int secfile = open(seckeyfilename, O_RDONLY);
 
584
      size_t buffer_capacity = 0;
 
585
      while(true){
 
586
        buffer_capacity = incbuffer((char **)&buffer.data,
 
587
                                    (size_t)buffer.size,
 
588
                                    (size_t)buffer_capacity);
 
589
        if(buffer_capacity == 0){
 
590
          perror_plus("incbuffer");
 
591
          free(buffer.data);
 
592
          buffer.data = NULL;
 
593
          break;
 
594
        }
 
595
        ssize_t bytes_read = read(secfile, buffer.data + buffer.size,
 
596
                                  BUFFER_SIZE);
 
597
        /* EOF */
 
598
        if(bytes_read == 0){
 
599
          break;
 
600
        }
 
601
        /* check bytes_read for failure */
 
602
        if(bytes_read < 0){
 
603
          perror_plus("read");
 
604
          free(buffer.data);
 
605
          buffer.data = NULL;
 
606
          break;
 
607
        }
 
608
        buffer.size += (unsigned int)bytes_read;
 
609
      }
 
610
      close(secfile);
 
611
    }
 
612
    /* If successful, use buffer to parse private key */
 
613
    gnutls_sec_param_t sec_param = GNUTLS_SEC_PARAM_ULTRA;
 
614
    if(buffer.data != NULL){
 
615
      {
 
616
        gnutls_openpgp_privkey_t privkey = NULL;
 
617
        ret = gnutls_openpgp_privkey_init(&privkey);
 
618
        if(ret != GNUTLS_E_SUCCESS){
 
619
          fprintf_plus(stderr, "Error initializing OpenPGP key"
 
620
                       " structure: %s", safer_gnutls_strerror(ret));
 
621
          free(buffer.data);
 
622
          buffer.data = NULL;
 
623
        } else {
 
624
          ret = gnutls_openpgp_privkey_import(privkey, &buffer,
 
625
                                            GNUTLS_OPENPGP_FMT_BASE64,
 
626
                                              "", 0);
 
627
          if(ret != GNUTLS_E_SUCCESS){
 
628
            fprintf_plus(stderr, "Error importing OpenPGP key : %s",
 
629
                         safer_gnutls_strerror(ret));
 
630
            privkey = NULL;
 
631
          }
 
632
          free(buffer.data);
 
633
          buffer.data = NULL;
 
634
          if(privkey != NULL){
 
635
            /* Use private key to suggest an appropriate sec_param */
 
636
            sec_param = gnutls_openpgp_privkey_sec_param(privkey);
 
637
            gnutls_openpgp_privkey_deinit(privkey);
 
638
            if(debug){
 
639
              fprintf_plus(stderr, "This OpenPGP key implies using a"
 
640
                           " GnuTLS security parameter \"%s\".\n",
 
641
                           safe_string(gnutls_sec_param_get_name
 
642
                                       (sec_param)));
 
643
            }
 
644
          }
 
645
        }
 
646
      }
 
647
      if(sec_param == GNUTLS_SEC_PARAM_UNKNOWN){
 
648
        /* Err on the side of caution */
 
649
        sec_param = GNUTLS_SEC_PARAM_ULTRA;
 
650
        if(debug){
 
651
          fprintf_plus(stderr, "Falling back to security parameter"
 
652
                       " \"%s\"\n",
 
653
                       safe_string(gnutls_sec_param_get_name
 
654
                                   (sec_param)));
 
655
        }
 
656
      }
 
657
    }
 
658
    uret = gnutls_sec_param_to_pk_bits(GNUTLS_PK_DH, sec_param);
 
659
    if(uret != 0){
 
660
      mc->dh_bits = uret;
 
661
      if(debug){
 
662
        fprintf_plus(stderr, "A \"%s\" GnuTLS security parameter"
 
663
                     " implies %u DH bits; using that.\n",
 
664
                     safe_string(gnutls_sec_param_get_name
 
665
                                 (sec_param)),
 
666
                     mc->dh_bits);
 
667
      }
 
668
    } else {
 
669
      fprintf_plus(stderr, "Failed to get implied number of DH"
 
670
                   " bits for security parameter \"%s\"): %s\n",
 
671
                   safe_string(gnutls_sec_param_get_name(sec_param)),
 
672
                   safer_gnutls_strerror(ret));
 
673
      goto globalfail;
 
674
    }
 
675
  } else if(debug){
 
676
    fprintf_plus(stderr, "DH bits explicitly set to %u\n",
 
677
                 mc->dh_bits);
 
678
  }
572
679
  ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
573
680
  if(ret != GNUTLS_E_SUCCESS){
574
 
    fprintf_plus(stderr, "Error in GnuTLS prime generation: %s\n",
575
 
                 safer_gnutls_strerror(ret));
 
681
    fprintf_plus(stderr, "Error in GnuTLS prime generation (%u bits):"
 
682
                 " %s\n", mc->dh_bits, safer_gnutls_strerror(ret));
576
683
    goto globalfail;
577
684
  }
578
685
  
650
757
static void empty_log(__attribute__((unused)) AvahiLogLevel level,
651
758
                      __attribute__((unused)) const char *txt){}
652
759
 
 
760
/* Set effective uid to 0, return errno */
 
761
__attribute__((warn_unused_result))
 
762
error_t raise_privileges(void){
 
763
  error_t old_errno = errno;
 
764
  error_t ret_errno = 0;
 
765
  if(seteuid(0) == -1){
 
766
    ret_errno = errno;
 
767
  }
 
768
  errno = old_errno;
 
769
  return ret_errno;
 
770
}
 
771
 
 
772
/* Set effective and real user ID to 0.  Return errno. */
 
773
__attribute__((warn_unused_result))
 
774
error_t raise_privileges_permanently(void){
 
775
  error_t old_errno = errno;
 
776
  error_t ret_errno = raise_privileges();
 
777
  if(ret_errno != 0){
 
778
    errno = old_errno;
 
779
    return ret_errno;
 
780
  }
 
781
  if(setuid(0) == -1){
 
782
    ret_errno = errno;
 
783
  }
 
784
  errno = old_errno;
 
785
  return ret_errno;
 
786
}
 
787
 
 
788
/* Set effective user ID to unprivileged saved user ID */
 
789
__attribute__((warn_unused_result))
 
790
error_t lower_privileges(void){
 
791
  error_t old_errno = errno;
 
792
  error_t ret_errno = 0;
 
793
  if(seteuid(uid) == -1){
 
794
    ret_errno = errno;
 
795
  }
 
796
  errno = old_errno;
 
797
  return ret_errno;
 
798
}
 
799
 
 
800
/* Lower privileges permanently */
 
801
__attribute__((warn_unused_result))
 
802
error_t lower_privileges_permanently(void){
 
803
  error_t old_errno = errno;
 
804
  error_t ret_errno = 0;
 
805
  if(setuid(uid) == -1){
 
806
    ret_errno = errno;
 
807
  }
 
808
  errno = old_errno;
 
809
  return ret_errno;
 
810
}
 
811
 
 
812
/* Helper function to add_local_route() and delete_local_route() */
 
813
__attribute__((nonnull, warn_unused_result))
 
814
static bool add_delete_local_route(const bool add,
 
815
                                   const char *address,
 
816
                                   AvahiIfIndex if_index){
 
817
  int ret;
 
818
  char helper[] = "mandos-client-iprouteadddel";
 
819
  char add_arg[] = "add";
 
820
  char delete_arg[] = "delete";
 
821
  char debug_flag[] = "--debug";
 
822
  char *pluginhelperdir = getenv("MANDOSPLUGINHELPERDIR");
 
823
  if(pluginhelperdir == NULL){
 
824
    if(debug){
 
825
      fprintf_plus(stderr, "MANDOSPLUGINHELPERDIR environment"
 
826
                   " variable not set; cannot run helper\n");
 
827
    }
 
828
    return false;
 
829
  }
 
830
  
 
831
  char interface[IF_NAMESIZE];
 
832
  if(if_indextoname((unsigned int)if_index, interface) == NULL){
 
833
    perror_plus("if_indextoname");
 
834
    return false;
 
835
  }
 
836
  
 
837
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
838
  if(devnull == -1){
 
839
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
840
    return false;
 
841
  }
 
842
  pid_t pid = fork();
 
843
  if(pid == 0){
 
844
    /* Child */
 
845
    /* Raise privileges */
 
846
    errno = raise_privileges_permanently();
 
847
    if(errno != 0){
 
848
      perror_plus("Failed to raise privileges");
 
849
      /* _exit(EX_NOPERM); */
 
850
    } else {
 
851
      /* Set group */
 
852
      errno = 0;
 
853
      ret = setgid(0);
 
854
      if(ret == -1){
 
855
        perror_plus("setgid");
 
856
        _exit(EX_NOPERM);
 
857
      }
 
858
      /* Reset supplementary groups */
 
859
      errno = 0;
 
860
      ret = setgroups(0, NULL);
 
861
      if(ret == -1){
 
862
        perror_plus("setgroups");
 
863
        _exit(EX_NOPERM);
 
864
      }
 
865
    }
 
866
    ret = dup2(devnull, STDIN_FILENO);
 
867
    if(ret == -1){
 
868
      perror_plus("dup2(devnull, STDIN_FILENO)");
 
869
      _exit(EX_OSERR);
 
870
    }
 
871
    ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
872
    if(ret == -1){
 
873
      perror_plus("close");
 
874
      _exit(EX_OSERR);
 
875
    }
 
876
    ret = dup2(STDERR_FILENO, STDOUT_FILENO);
 
877
    if(ret == -1){
 
878
      perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
 
879
      _exit(EX_OSERR);
 
880
    }
 
881
    int helperdir_fd = (int)TEMP_FAILURE_RETRY(open(pluginhelperdir,
 
882
                                                    O_RDONLY
 
883
                                                    | O_DIRECTORY
 
884
                                                    | O_PATH
 
885
                                                    | O_CLOEXEC));
 
886
    if(helperdir_fd == -1){
 
887
      perror_plus("open");
 
888
      _exit(EX_UNAVAILABLE);
 
889
    }
 
890
    int helper_fd = (int)TEMP_FAILURE_RETRY(openat(helperdir_fd,
 
891
                                                   helper, O_RDONLY));
 
892
    if(helper_fd == -1){
 
893
      perror_plus("openat");
 
894
      _exit(EX_UNAVAILABLE);
 
895
    }
 
896
    TEMP_FAILURE_RETRY(close(helperdir_fd));
 
897
#ifdef __GNUC__
 
898
#pragma GCC diagnostic push
 
899
#pragma GCC diagnostic ignored "-Wcast-qual"
 
900
#endif
 
901
    if(fexecve(helper_fd, (char *const [])
 
902
               { helper, add ? add_arg : delete_arg, (char *)address,
 
903
                   interface, debug ? debug_flag : NULL, NULL },
 
904
               environ) == -1){
 
905
#ifdef __GNUC__
 
906
#pragma GCC diagnostic pop
 
907
#endif
 
908
      perror_plus("fexecve");
 
909
      _exit(EXIT_FAILURE);
 
910
    }
 
911
  }
 
912
  if(pid == -1){
 
913
    perror_plus("fork");
 
914
    return false;
 
915
  }
 
916
  int status;
 
917
  pid_t pret = -1;
 
918
  errno = 0;
 
919
  do {
 
920
    pret = waitpid(pid, &status, 0);
 
921
    if(pret == -1 and errno == EINTR and quit_now){
 
922
      int errno_raising = 0;
 
923
      if((errno = raise_privileges()) != 0){
 
924
        errno_raising = errno;
 
925
        perror_plus("Failed to raise privileges in order to"
 
926
                    " kill helper program");
 
927
      }
 
928
      if(kill(pid, SIGTERM) == -1){
 
929
        perror_plus("kill");
 
930
      }
 
931
      if((errno_raising == 0) and (errno = lower_privileges()) != 0){
 
932
        perror_plus("Failed to lower privileges after killing"
 
933
                    " helper program");
 
934
      }
 
935
      return false;
 
936
    }
 
937
  } while(pret == -1 and errno == EINTR);
 
938
  if(pret == -1){
 
939
    perror_plus("waitpid");
 
940
    return false;
 
941
  }
 
942
  if(WIFEXITED(status)){
 
943
    if(WEXITSTATUS(status) != 0){
 
944
      fprintf_plus(stderr, "Error: iprouteadddel exited"
 
945
                   " with status %d\n", WEXITSTATUS(status));
 
946
      return false;
 
947
    }
 
948
    return true;
 
949
  }
 
950
  if(WIFSIGNALED(status)){
 
951
    fprintf_plus(stderr, "Error: iprouteadddel died by"
 
952
                 " signal %d\n", WTERMSIG(status));
 
953
    return false;
 
954
  }
 
955
  fprintf_plus(stderr, "Error: iprouteadddel crashed\n");
 
956
  return false;
 
957
}
 
958
 
 
959
__attribute__((nonnull, warn_unused_result))
 
960
static bool add_local_route(const char *address,
 
961
                            AvahiIfIndex if_index){
 
962
  if(debug){
 
963
    fprintf_plus(stderr, "Adding route to %s\n", address);
 
964
  }
 
965
  return add_delete_local_route(true, address, if_index);
 
966
}
 
967
 
 
968
__attribute__((nonnull, warn_unused_result))
 
969
static bool delete_local_route(const char *address,
 
970
                               AvahiIfIndex if_index){
 
971
  if(debug){
 
972
    fprintf_plus(stderr, "Removing route to %s\n", address);
 
973
  }
 
974
  return add_delete_local_route(false, address, if_index);
 
975
}
 
976
 
653
977
/* Called when a Mandos server is found */
654
978
__attribute__((nonnull, warn_unused_result))
655
979
static int start_mandos_communication(const char *ip, in_port_t port,
666
990
  int retval = -1;
667
991
  gnutls_session_t session;
668
992
  int pf;                       /* Protocol family */
 
993
  bool route_added = false;
669
994
  
670
995
  errno = 0;
671
996
  
729
1054
                 PRIuMAX "\n", ip, (uintmax_t)port);
730
1055
  }
731
1056
  
732
 
  tcp_sd = socket(pf, SOCK_STREAM, 0);
 
1057
  tcp_sd = socket(pf, SOCK_STREAM | SOCK_CLOEXEC, 0);
733
1058
  if(tcp_sd < 0){
734
1059
    int e = errno;
735
1060
    perror_plus("socket");
824
1149
    goto mandos_end;
825
1150
  }
826
1151
  
827
 
  if(af == AF_INET6){
828
 
    ret = connect(tcp_sd, (struct sockaddr *)&to,
829
 
                  sizeof(struct sockaddr_in6));
830
 
  } else {
831
 
    ret = connect(tcp_sd, (struct sockaddr *)&to, /* IPv4 */
832
 
                  sizeof(struct sockaddr_in));
833
 
  }
834
 
  if(ret < 0){
835
 
    if((errno != ECONNREFUSED and errno != ENETUNREACH) or debug){
836
 
      int e = errno;
837
 
      perror_plus("connect");
838
 
      errno = e;
839
 
    }
840
 
    goto mandos_end;
841
 
  }
842
 
  
843
 
  if(quit_now){
844
 
    errno = EINTR;
845
 
    goto mandos_end;
 
1152
  while(true){
 
1153
    if(af == AF_INET6){
 
1154
      ret = connect(tcp_sd, (struct sockaddr *)&to,
 
1155
                    sizeof(struct sockaddr_in6));
 
1156
    } else {
 
1157
      ret = connect(tcp_sd, (struct sockaddr *)&to, /* IPv4 */
 
1158
                    sizeof(struct sockaddr_in));
 
1159
    }
 
1160
    if(ret < 0){
 
1161
      if(errno == ENETUNREACH
 
1162
         and if_index != AVAHI_IF_UNSPEC
 
1163
         and connect_to == NULL
 
1164
         and not route_added and
 
1165
         ((af == AF_INET6 and not
 
1166
           IN6_IS_ADDR_LINKLOCAL(&(((struct sockaddr_in6 *)
 
1167
                                    &to)->sin6_addr)))
 
1168
          or (af == AF_INET and
 
1169
              /* Not a a IPv4LL address */
 
1170
              (ntohl(((struct sockaddr_in *)&to)->sin_addr.s_addr)
 
1171
               & 0xFFFF0000L) != 0xA9FE0000L))){
 
1172
        /* Work around Avahi bug - Avahi does not announce link-local
 
1173
           addresses if it has a global address, so local hosts with
 
1174
           *only* a link-local address (e.g. Mandos clients) cannot
 
1175
           connect to a Mandos server announced by Avahi on a server
 
1176
           host with a global address.  Work around this by retrying
 
1177
           with an explicit route added with the server's address.
 
1178
           
 
1179
           Avahi bug reference:
 
1180
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1181
           https://bugs.debian.org/587961
 
1182
        */
 
1183
        if(debug){
 
1184
          fprintf_plus(stderr, "Mandos server unreachable, trying"
 
1185
                       " direct route\n");
 
1186
        }
 
1187
        int e = errno;
 
1188
        route_added = add_local_route(ip, if_index);
 
1189
        if(route_added){
 
1190
          continue;
 
1191
        }
 
1192
        errno = e;
 
1193
      }
 
1194
      if(errno != ECONNREFUSED or debug){
 
1195
        int e = errno;
 
1196
        perror_plus("connect");
 
1197
        errno = e;
 
1198
      }
 
1199
      goto mandos_end;
 
1200
    }
 
1201
    
 
1202
    if(quit_now){
 
1203
      errno = EINTR;
 
1204
      goto mandos_end;
 
1205
    }
 
1206
    break;
846
1207
  }
847
1208
  
848
1209
  const char *out = mandos_protocol_version;
1031
1392
  
1032
1393
 mandos_end:
1033
1394
  {
 
1395
    if(route_added){
 
1396
      if(not delete_local_route(ip, if_index)){
 
1397
        fprintf_plus(stderr, "Failed to delete local route to %s on"
 
1398
                     " interface %d", ip, if_index);
 
1399
      }
 
1400
    }
1034
1401
    int e = errno;
1035
1402
    free(decrypted_buffer);
1036
1403
    free(buffer);
1462
1829
  }
1463
1830
}
1464
1831
 
1465
 
/* Set effective uid to 0, return errno */
1466
 
__attribute__((warn_unused_result))
1467
 
error_t raise_privileges(void){
1468
 
  error_t old_errno = errno;
1469
 
  error_t ret_errno = 0;
1470
 
  if(seteuid(0) == -1){
1471
 
    ret_errno = errno;
1472
 
  }
1473
 
  errno = old_errno;
1474
 
  return ret_errno;
1475
 
}
1476
 
 
1477
 
/* Set effective and real user ID to 0.  Return errno. */
1478
 
__attribute__((warn_unused_result))
1479
 
error_t raise_privileges_permanently(void){
1480
 
  error_t old_errno = errno;
1481
 
  error_t ret_errno = raise_privileges();
1482
 
  if(ret_errno != 0){
1483
 
    errno = old_errno;
1484
 
    return ret_errno;
1485
 
  }
1486
 
  if(setuid(0) == -1){
1487
 
    ret_errno = errno;
1488
 
  }
1489
 
  errno = old_errno;
1490
 
  return ret_errno;
1491
 
}
1492
 
 
1493
 
/* Set effective user ID to unprivileged saved user ID */
1494
 
__attribute__((warn_unused_result))
1495
 
error_t lower_privileges(void){
1496
 
  error_t old_errno = errno;
1497
 
  error_t ret_errno = 0;
1498
 
  if(seteuid(uid) == -1){
1499
 
    ret_errno = errno;
1500
 
  }
1501
 
  errno = old_errno;
1502
 
  return ret_errno;
1503
 
}
1504
 
 
1505
 
/* Lower privileges permanently */
1506
 
__attribute__((warn_unused_result))
1507
 
error_t lower_privileges_permanently(void){
1508
 
  error_t old_errno = errno;
1509
 
  error_t ret_errno = 0;
1510
 
  if(setuid(uid) == -1){
1511
 
    ret_errno = errno;
1512
 
  }
1513
 
  errno = old_errno;
1514
 
  return ret_errno;
1515
 
}
1516
 
 
1517
1832
__attribute__((nonnull))
1518
1833
void run_network_hooks(const char *mode, const char *interface,
1519
1834
                       const float delay){
1520
1835
  struct dirent **direntries = NULL;
1521
1836
  if(hookdir_fd == -1){
1522
 
    hookdir_fd = open(hookdir, O_RDONLY);
 
1837
    hookdir_fd = open(hookdir, O_RDONLY | O_DIRECTORY | O_PATH
 
1838
                      | O_CLOEXEC);
1523
1839
    if(hookdir_fd == -1){
1524
1840
      if(errno == ENOENT){
1525
1841
        if(debug){
1550
1866
  }
1551
1867
  struct dirent *direntry;
1552
1868
  int ret;
1553
 
  int devnull = open("/dev/null", O_RDONLY);
 
1869
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1870
  if(devnull == -1){
 
1871
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1872
    return;
 
1873
  }
1554
1874
  for(int i = 0; i < numhooks; i++){
1555
1875
    direntry = direntries[i];
1556
1876
    if(debug){
1580
1900
        perror_plus("setgroups");
1581
1901
        _exit(EX_NOPERM);
1582
1902
      }
1583
 
      ret = dup2(devnull, STDIN_FILENO);
1584
 
      if(ret == -1){
1585
 
        perror_plus("dup2(devnull, STDIN_FILENO)");
1586
 
        _exit(EX_OSERR);
1587
 
      }
1588
 
      ret = close(devnull);
1589
 
      if(ret == -1){
1590
 
        perror_plus("close");
1591
 
        _exit(EX_OSERR);
1592
 
      }
1593
 
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
1594
 
      if(ret == -1){
1595
 
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
1596
 
        _exit(EX_OSERR);
1597
 
      }
1598
1903
      ret = setenv("MANDOSNETHOOKDIR", hookdir, 1);
1599
1904
      if(ret == -1){
1600
1905
        perror_plus("setenv");
1635
1940
          _exit(EX_OSERR);
1636
1941
        }
1637
1942
      }
1638
 
      int hook_fd = openat(hookdir_fd, direntry->d_name, O_RDONLY);
 
1943
      int hook_fd = (int)TEMP_FAILURE_RETRY(openat(hookdir_fd,
 
1944
                                                   direntry->d_name,
 
1945
                                                   O_RDONLY));
1639
1946
      if(hook_fd == -1){
1640
1947
        perror_plus("openat");
1641
1948
        _exit(EXIT_FAILURE);
1644
1951
        perror_plus("close");
1645
1952
        _exit(EXIT_FAILURE);
1646
1953
      }
 
1954
      ret = dup2(devnull, STDIN_FILENO);
 
1955
      if(ret == -1){
 
1956
        perror_plus("dup2(devnull, STDIN_FILENO)");
 
1957
        _exit(EX_OSERR);
 
1958
      }
 
1959
      ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
1960
      if(ret == -1){
 
1961
        perror_plus("close");
 
1962
        _exit(EX_OSERR);
 
1963
      }
 
1964
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
 
1965
      if(ret == -1){
 
1966
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
 
1967
        _exit(EX_OSERR);
 
1968
      }
1647
1969
      if(fexecve(hook_fd, (char *const []){ direntry->d_name, NULL },
1648
1970
                 environ) == -1){
1649
1971
        perror_plus("fexecve");
1900
2222
}
1901
2223
 
1902
2224
int main(int argc, char *argv[]){
1903
 
  mandos_context mc = { .server = NULL, .dh_bits = 1024,
 
2225
  mandos_context mc = { .server = NULL, .dh_bits = 0,
1904
2226
                        .priority = "SECURE256:!CTYPE-X.509:"
1905
 
                        "+CTYPE-OPENPGP", .current_server = NULL,
 
2227
                        "+CTYPE-OPENPGP:!RSA", .current_server = NULL,
1906
2228
                        .interfaces = NULL, .interfaces_size = 0 };
1907
2229
  AvahiSServiceBrowser *sb = NULL;
1908
2230
  error_t ret_errno;
2097
2419
      goto end;
2098
2420
    }
2099
2421
  }
2100
 
    
 
2422
  
2101
2423
  {
2102
2424
    /* Work around Debian bug #633582:
2103
2425
       <http://bugs.debian.org/633582> */
2130
2452
          TEMP_FAILURE_RETRY(close(seckey_fd));
2131
2453
        }
2132
2454
      }
2133
 
    
 
2455
      
2134
2456
      if(strcmp(pubkey, PATHDIR "/" PUBKEY) == 0){
2135
2457
        int pubkey_fd = open(pubkey, O_RDONLY);
2136
2458
        if(pubkey_fd == -1){
2151
2473
          TEMP_FAILURE_RETRY(close(pubkey_fd));
2152
2474
        }
2153
2475
      }
2154
 
    
 
2476
      
2155
2477
      /* Lower privileges */
2156
2478
      ret_errno = lower_privileges();
2157
2479
      if(ret_errno != 0){
2626
2948
  /* Removes the GPGME temp directory and all files inside */
2627
2949
  if(tempdir != NULL){
2628
2950
    struct dirent **direntries = NULL;
2629
 
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY |
2630
 
                                                  O_NOFOLLOW));
 
2951
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
 
2952
                                                  | O_NOFOLLOW
 
2953
                                                  | O_DIRECTORY
 
2954
                                                  | O_PATH));
2631
2955
    if(tempdir_fd == -1){
2632
2956
      perror_plus("open");
2633
2957
    } else {