/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: 2015-07-06 20:09:47 UTC
  • mto: This revision was merged to the branch mainline in revision 759.
  • Revision ID: teddy@recompile.se-20150706200947-w21u4eq74efgl6r5
Fix minor bugs and typos and add some more debug output.

* Makefile (install-client-nokey): Create plugin-helpers directory and
                                   the mandos-client-iprouteadddel
                                   helper program.
* initramfs-tools-hook (PLUGINHELPERDIR): Fix typo.
* plugins.d/mandos-client.c: Change terminology; routes are "deleted",
                             not "removed".  All occurences changed.
  (add_remove_local_route): Renamed to "add_delete_local_route".  All
                            callers changed.  Also pass "--debug" flag
                            to helper if in debug mode.
  (add_local_route): Add debugging output.
  (remove_local_route): Renamed to "delete_local_route".  All callers
                        changed.  Also pass "--debug" flag to helper
                        if in debug mode.
  (start_mandos_communication): Add debug output when adding route.

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-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
 
 
503
496
__attribute__((warn_unused_result))
504
497
static const char *safer_gnutls_strerror(int value){
505
498
  const char *ret = gnutls_strerror(value);
506
 
  return safe_string(ret);
 
499
  if(ret == NULL)
 
500
    ret = "(unknown)";
 
501
  return ret;
507
502
}
508
503
 
509
504
/* GnuTLS log function callback */
518
513
                              const char *seckeyfilename,
519
514
                              mandos_context *mc){
520
515
  int ret;
521
 
  unsigned int uret;
522
516
  
523
517
  if(debug){
524
518
    fprintf_plus(stderr, "Initializing GnuTLS\n");
575
569
                 safer_gnutls_strerror(ret));
576
570
    goto globalfail;
577
571
  }
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
 
  }
679
572
  ret = gnutls_dh_params_generate2(mc->dh_params, mc->dh_bits);
680
573
  if(ret != GNUTLS_E_SUCCESS){
681
 
    fprintf_plus(stderr, "Error in GnuTLS prime generation (%u bits):"
682
 
                 " %s\n", mc->dh_bits, safer_gnutls_strerror(ret));
 
574
    fprintf_plus(stderr, "Error in GnuTLS prime generation: %s\n",
 
575
                 safer_gnutls_strerror(ret));
683
576
    goto globalfail;
684
577
  }
685
578
  
757
650
static void empty_log(__attribute__((unused)) AvahiLogLevel level,
758
651
                      __attribute__((unused)) const char *txt){}
759
652
 
 
653
/* Set effective uid to 0, return errno */
 
654
__attribute__((warn_unused_result))
 
655
error_t raise_privileges(void){
 
656
  error_t old_errno = errno;
 
657
  error_t ret_errno = 0;
 
658
  if(seteuid(0) == -1){
 
659
    ret_errno = errno;
 
660
  }
 
661
  errno = old_errno;
 
662
  return ret_errno;
 
663
}
 
664
 
 
665
/* Set effective and real user ID to 0.  Return errno. */
 
666
__attribute__((warn_unused_result))
 
667
error_t raise_privileges_permanently(void){
 
668
  error_t old_errno = errno;
 
669
  error_t ret_errno = raise_privileges();
 
670
  if(ret_errno != 0){
 
671
    errno = old_errno;
 
672
    return ret_errno;
 
673
  }
 
674
  if(setuid(0) == -1){
 
675
    ret_errno = errno;
 
676
  }
 
677
  errno = old_errno;
 
678
  return ret_errno;
 
679
}
 
680
 
 
681
/* Set effective user ID to unprivileged saved user ID */
 
682
__attribute__((warn_unused_result))
 
683
error_t lower_privileges(void){
 
684
  error_t old_errno = errno;
 
685
  error_t ret_errno = 0;
 
686
  if(seteuid(uid) == -1){
 
687
    ret_errno = errno;
 
688
  }
 
689
  errno = old_errno;
 
690
  return ret_errno;
 
691
}
 
692
 
 
693
/* Lower privileges permanently */
 
694
__attribute__((warn_unused_result))
 
695
error_t lower_privileges_permanently(void){
 
696
  error_t old_errno = errno;
 
697
  error_t ret_errno = 0;
 
698
  if(setuid(uid) == -1){
 
699
    ret_errno = errno;
 
700
  }
 
701
  errno = old_errno;
 
702
  return ret_errno;
 
703
}
 
704
 
 
705
/* Helper function to add_local_route() and delete_local_route() */
 
706
__attribute__((nonnull, warn_unused_result))
 
707
static bool add_delete_local_route(const bool add,
 
708
                                   const char *address,
 
709
                                   AvahiIfIndex if_index){
 
710
  int ret;
 
711
  char helper[] = "mandos-client-iprouteadddel";
 
712
  char add_arg[] = "add";
 
713
  char delete_arg[] = "delete";
 
714
  char debug_flag[] = "--debug";
 
715
  char *pluginhelperdir = getenv("MANDOSPLUGINHELPERDIR");
 
716
  if(pluginhelperdir == NULL){
 
717
    if(debug){
 
718
      fprintf_plus(stderr, "MANDOSPLUGINHELPERDIR environment"
 
719
                   " variable not set; cannot run helper\n");
 
720
    }
 
721
    return false;
 
722
  }
 
723
  
 
724
  char interface[IF_NAMESIZE];
 
725
  if(if_indextoname((unsigned int)if_index, interface) == NULL){
 
726
    perror_plus("if_indextoname");
 
727
    return false;
 
728
  }
 
729
  
 
730
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
731
  if(devnull == -1){
 
732
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
733
    return false;
 
734
  }
 
735
  pid_t pid = fork();
 
736
  if(pid == 0){
 
737
    /* Child */
 
738
    /* Raise privileges */
 
739
    errno = raise_privileges_permanently();
 
740
    if(errno != 0){
 
741
      perror_plus("Failed to raise privileges");
 
742
      /* _exit(EX_NOPERM); */
 
743
    } else {
 
744
      /* Set group */
 
745
      errno = 0;
 
746
      ret = setgid(0);
 
747
      if(ret == -1){
 
748
        perror_plus("setgid");
 
749
        _exit(EX_NOPERM);
 
750
      }
 
751
      /* Reset supplementary groups */
 
752
      errno = 0;
 
753
      ret = setgroups(0, NULL);
 
754
      if(ret == -1){
 
755
        perror_plus("setgroups");
 
756
        _exit(EX_NOPERM);
 
757
      }
 
758
    }
 
759
    ret = dup2(devnull, STDIN_FILENO);
 
760
    if(ret == -1){
 
761
      perror_plus("dup2(devnull, STDIN_FILENO)");
 
762
      _exit(EX_OSERR);
 
763
    }
 
764
    ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
765
    if(ret == -1){
 
766
      perror_plus("close");
 
767
      _exit(EX_OSERR);
 
768
    }
 
769
    ret = dup2(STDERR_FILENO, STDOUT_FILENO);
 
770
    if(ret == -1){
 
771
      perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
 
772
      _exit(EX_OSERR);
 
773
    }
 
774
    int helperdir_fd = (int)TEMP_FAILURE_RETRY(open(pluginhelperdir,
 
775
                                                    O_RDONLY
 
776
                                                    | O_DIRECTORY
 
777
                                                    | O_PATH
 
778
                                                    | O_CLOEXEC));
 
779
    if(helperdir_fd == -1){
 
780
      perror_plus("open");
 
781
      _exit(EX_UNAVAILABLE);
 
782
    }
 
783
    int helper_fd = (int)TEMP_FAILURE_RETRY(openat(helperdir_fd,
 
784
                                                   helper, O_RDONLY));
 
785
    if(helper_fd == -1){
 
786
      perror_plus("openat");
 
787
      _exit(EX_UNAVAILABLE);
 
788
    }
 
789
    TEMP_FAILURE_RETRY(close(helperdir_fd));
 
790
#ifdef __GNUC__
 
791
#pragma GCC diagnostic push
 
792
#pragma GCC diagnostic ignored "-Wcast-qual"
 
793
#endif
 
794
    if(fexecve(helper_fd, (char *const [])
 
795
               { helper, add ? add_arg : delete_arg, (char *)address,
 
796
                   interface, debug ? debug_flag : NULL, NULL },
 
797
               environ) == -1){
 
798
#ifdef __GNUC__
 
799
#pragma GCC diagnostic pop
 
800
#endif
 
801
      perror_plus("fexecve");
 
802
      _exit(EXIT_FAILURE);
 
803
    }
 
804
  }
 
805
  if(pid == -1){
 
806
    perror_plus("fork");
 
807
    return false;
 
808
  }
 
809
  int status;
 
810
  pid_t pret = -1;
 
811
  errno = 0;
 
812
  do {
 
813
    pret = waitpid(pid, &status, 0);
 
814
    if(pret == -1 and errno == EINTR and quit_now){
 
815
      int errno_raising = 0;
 
816
      if((errno = raise_privileges()) != 0){
 
817
        errno_raising = errno;
 
818
        perror_plus("Failed to raise privileges in order to"
 
819
                    " kill helper program");
 
820
      }
 
821
      if(kill(pid, SIGTERM) == -1){
 
822
        perror_plus("kill");
 
823
      }
 
824
      if((errno_raising == 0) and (errno = lower_privileges()) != 0){
 
825
        perror_plus("Failed to lower privileges after killing"
 
826
                    " helper program");
 
827
      }
 
828
      return false;
 
829
    }
 
830
  } while(pret == -1 and errno == EINTR);
 
831
  if(pret == -1){
 
832
    perror_plus("waitpid");
 
833
    return false;
 
834
  }
 
835
  if(WIFEXITED(status)){
 
836
    if(WEXITSTATUS(status) != 0){
 
837
      fprintf_plus(stderr, "Error: iprouteadddel exited"
 
838
                   " with status %d\n", WEXITSTATUS(status));
 
839
      return false;
 
840
    }
 
841
    return true;
 
842
  }
 
843
  if(WIFSIGNALED(status)){
 
844
    fprintf_plus(stderr, "Error: iprouteadddel died by"
 
845
                 " signal %d\n", WTERMSIG(status));
 
846
    return false;
 
847
  }
 
848
  fprintf_plus(stderr, "Error: iprouteadddel crashed\n");
 
849
  return false;
 
850
}
 
851
 
 
852
__attribute__((nonnull, warn_unused_result))
 
853
static bool add_local_route(const char *address,
 
854
                            AvahiIfIndex if_index){
 
855
  if(debug){
 
856
    fprintf_plus(stderr, "Adding route to %s\n", address);
 
857
  }
 
858
  return add_delete_local_route(true, address, if_index);
 
859
}
 
860
 
 
861
__attribute__((nonnull, warn_unused_result))
 
862
static bool delete_local_route(const char *address,
 
863
                               AvahiIfIndex if_index){
 
864
  if(debug){
 
865
    fprintf_plus(stderr, "Removing route to %s\n", address);
 
866
  }
 
867
  return add_delete_local_route(false, address, if_index);
 
868
}
 
869
 
760
870
/* Called when a Mandos server is found */
761
871
__attribute__((nonnull, warn_unused_result))
762
872
static int start_mandos_communication(const char *ip, in_port_t port,
773
883
  int retval = -1;
774
884
  gnutls_session_t session;
775
885
  int pf;                       /* Protocol family */
 
886
  bool route_added = false;
776
887
  
777
888
  errno = 0;
778
889
  
836
947
                 PRIuMAX "\n", ip, (uintmax_t)port);
837
948
  }
838
949
  
839
 
  tcp_sd = socket(pf, SOCK_STREAM, 0);
 
950
  tcp_sd = socket(pf, SOCK_STREAM | SOCK_CLOEXEC, 0);
840
951
  if(tcp_sd < 0){
841
952
    int e = errno;
842
953
    perror_plus("socket");
931
1042
    goto mandos_end;
932
1043
  }
933
1044
  
934
 
  if(af == AF_INET6){
935
 
    ret = connect(tcp_sd, (struct sockaddr *)&to,
936
 
                  sizeof(struct sockaddr_in6));
937
 
  } else {
938
 
    ret = connect(tcp_sd, (struct sockaddr *)&to, /* IPv4 */
939
 
                  sizeof(struct sockaddr_in));
940
 
  }
941
 
  if(ret < 0){
942
 
    if((errno != ECONNREFUSED and errno != ENETUNREACH) or debug){
943
 
      int e = errno;
944
 
      perror_plus("connect");
945
 
      errno = e;
946
 
    }
947
 
    goto mandos_end;
948
 
  }
949
 
  
950
 
  if(quit_now){
951
 
    errno = EINTR;
952
 
    goto mandos_end;
 
1045
  while(true){
 
1046
    if(af == AF_INET6){
 
1047
      ret = connect(tcp_sd, (struct sockaddr *)&to,
 
1048
                    sizeof(struct sockaddr_in6));
 
1049
    } else {
 
1050
      ret = connect(tcp_sd, (struct sockaddr *)&to, /* IPv4 */
 
1051
                    sizeof(struct sockaddr_in));
 
1052
    }
 
1053
    if(ret < 0){
 
1054
      if(errno == ENETUNREACH
 
1055
         and if_index != AVAHI_IF_UNSPEC
 
1056
         and connect_to == NULL
 
1057
         and not route_added and
 
1058
         ((af == AF_INET6 and not
 
1059
           IN6_IS_ADDR_LINKLOCAL(&(((struct sockaddr_in6 *)
 
1060
                                    &to)->sin6_addr)))
 
1061
          or (af == AF_INET and
 
1062
              /* Not a a IPv4LL address */
 
1063
              (ntohl(((struct sockaddr_in *)&to)->sin_addr.s_addr)
 
1064
               & 0xFFFF0000L) != 0xA9FE0000L))){
 
1065
        /* Work around Avahi bug - Avahi does not announce link-local
 
1066
           addresses if it has a global address, so local hosts with
 
1067
           *only* a link-local address (e.g. Mandos clients) cannot
 
1068
           connect to a Mandos server announced by Avahi on a server
 
1069
           host with a global address.  Work around this by retrying
 
1070
           with an explicit route added with the server's address.
 
1071
           
 
1072
           Avahi bug reference:
 
1073
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1074
           https://bugs.debian.org/587961
 
1075
        */
 
1076
        if(debug){
 
1077
          fprintf_plus(stderr, "Mandos server unreachable, trying"
 
1078
                       " direct route\n");
 
1079
        }
 
1080
        int e = errno;
 
1081
        route_added = add_local_route(ip, if_index);
 
1082
        if(route_added){
 
1083
          continue;
 
1084
        }
 
1085
        errno = e;
 
1086
      }
 
1087
      if(errno != ECONNREFUSED or debug){
 
1088
        int e = errno;
 
1089
        perror_plus("connect");
 
1090
        errno = e;
 
1091
      }
 
1092
      goto mandos_end;
 
1093
    }
 
1094
    
 
1095
    if(quit_now){
 
1096
      errno = EINTR;
 
1097
      goto mandos_end;
 
1098
    }
 
1099
    break;
953
1100
  }
954
1101
  
955
1102
  const char *out = mandos_protocol_version;
1138
1285
  
1139
1286
 mandos_end:
1140
1287
  {
 
1288
    if(route_added){
 
1289
      if(not delete_local_route(ip, if_index)){
 
1290
        fprintf_plus(stderr, "Failed to delete local route to %s on"
 
1291
                     " interface %d", ip, if_index);
 
1292
      }
 
1293
    }
1141
1294
    int e = errno;
1142
1295
    free(decrypted_buffer);
1143
1296
    free(buffer);
1569
1722
  }
1570
1723
}
1571
1724
 
1572
 
/* Set effective uid to 0, return errno */
1573
 
__attribute__((warn_unused_result))
1574
 
error_t raise_privileges(void){
1575
 
  error_t old_errno = errno;
1576
 
  error_t ret_errno = 0;
1577
 
  if(seteuid(0) == -1){
1578
 
    ret_errno = errno;
1579
 
  }
1580
 
  errno = old_errno;
1581
 
  return ret_errno;
1582
 
}
1583
 
 
1584
 
/* Set effective and real user ID to 0.  Return errno. */
1585
 
__attribute__((warn_unused_result))
1586
 
error_t raise_privileges_permanently(void){
1587
 
  error_t old_errno = errno;
1588
 
  error_t ret_errno = raise_privileges();
1589
 
  if(ret_errno != 0){
1590
 
    errno = old_errno;
1591
 
    return ret_errno;
1592
 
  }
1593
 
  if(setuid(0) == -1){
1594
 
    ret_errno = errno;
1595
 
  }
1596
 
  errno = old_errno;
1597
 
  return ret_errno;
1598
 
}
1599
 
 
1600
 
/* Set effective user ID to unprivileged saved user ID */
1601
 
__attribute__((warn_unused_result))
1602
 
error_t lower_privileges(void){
1603
 
  error_t old_errno = errno;
1604
 
  error_t ret_errno = 0;
1605
 
  if(seteuid(uid) == -1){
1606
 
    ret_errno = errno;
1607
 
  }
1608
 
  errno = old_errno;
1609
 
  return ret_errno;
1610
 
}
1611
 
 
1612
 
/* Lower privileges permanently */
1613
 
__attribute__((warn_unused_result))
1614
 
error_t lower_privileges_permanently(void){
1615
 
  error_t old_errno = errno;
1616
 
  error_t ret_errno = 0;
1617
 
  if(setuid(uid) == -1){
1618
 
    ret_errno = errno;
1619
 
  }
1620
 
  errno = old_errno;
1621
 
  return ret_errno;
1622
 
}
1623
 
 
1624
1725
__attribute__((nonnull))
1625
1726
void run_network_hooks(const char *mode, const char *interface,
1626
1727
                       const float delay){
1627
1728
  struct dirent **direntries = NULL;
1628
1729
  if(hookdir_fd == -1){
1629
 
    hookdir_fd = open(hookdir, O_RDONLY);
 
1730
    hookdir_fd = open(hookdir, O_RDONLY | O_DIRECTORY | O_PATH
 
1731
                      | O_CLOEXEC);
1630
1732
    if(hookdir_fd == -1){
1631
1733
      if(errno == ENOENT){
1632
1734
        if(debug){
1657
1759
  }
1658
1760
  struct dirent *direntry;
1659
1761
  int ret;
1660
 
  int devnull = open("/dev/null", O_RDONLY);
 
1762
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1763
  if(devnull == -1){
 
1764
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1765
    return;
 
1766
  }
1661
1767
  for(int i = 0; i < numhooks; i++){
1662
1768
    direntry = direntries[i];
1663
1769
    if(debug){
1687
1793
        perror_plus("setgroups");
1688
1794
        _exit(EX_NOPERM);
1689
1795
      }
1690
 
      ret = dup2(devnull, STDIN_FILENO);
1691
 
      if(ret == -1){
1692
 
        perror_plus("dup2(devnull, STDIN_FILENO)");
1693
 
        _exit(EX_OSERR);
1694
 
      }
1695
 
      ret = close(devnull);
1696
 
      if(ret == -1){
1697
 
        perror_plus("close");
1698
 
        _exit(EX_OSERR);
1699
 
      }
1700
 
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
1701
 
      if(ret == -1){
1702
 
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
1703
 
        _exit(EX_OSERR);
1704
 
      }
1705
1796
      ret = setenv("MANDOSNETHOOKDIR", hookdir, 1);
1706
1797
      if(ret == -1){
1707
1798
        perror_plus("setenv");
1742
1833
          _exit(EX_OSERR);
1743
1834
        }
1744
1835
      }
1745
 
      int hook_fd = openat(hookdir_fd, direntry->d_name, O_RDONLY);
 
1836
      int hook_fd = (int)TEMP_FAILURE_RETRY(openat(hookdir_fd,
 
1837
                                                   direntry->d_name,
 
1838
                                                   O_RDONLY));
1746
1839
      if(hook_fd == -1){
1747
1840
        perror_plus("openat");
1748
1841
        _exit(EXIT_FAILURE);
1751
1844
        perror_plus("close");
1752
1845
        _exit(EXIT_FAILURE);
1753
1846
      }
 
1847
      ret = dup2(devnull, STDIN_FILENO);
 
1848
      if(ret == -1){
 
1849
        perror_plus("dup2(devnull, STDIN_FILENO)");
 
1850
        _exit(EX_OSERR);
 
1851
      }
 
1852
      ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
1853
      if(ret == -1){
 
1854
        perror_plus("close");
 
1855
        _exit(EX_OSERR);
 
1856
      }
 
1857
      ret = dup2(STDERR_FILENO, STDOUT_FILENO);
 
1858
      if(ret == -1){
 
1859
        perror_plus("dup2(STDERR_FILENO, STDOUT_FILENO)");
 
1860
        _exit(EX_OSERR);
 
1861
      }
1754
1862
      if(fexecve(hook_fd, (char *const []){ direntry->d_name, NULL },
1755
1863
                 environ) == -1){
1756
1864
        perror_plus("fexecve");
2007
2115
}
2008
2116
 
2009
2117
int main(int argc, char *argv[]){
2010
 
  mandos_context mc = { .server = NULL, .dh_bits = 0,
 
2118
  mandos_context mc = { .server = NULL, .dh_bits = 1024,
2011
2119
                        .priority = "SECURE256:!CTYPE-X.509:"
2012
 
                        "+CTYPE-OPENPGP:!RSA", .current_server = NULL,
 
2120
                        "+CTYPE-OPENPGP", .current_server = NULL,
2013
2121
                        .interfaces = NULL, .interfaces_size = 0 };
2014
2122
  AvahiSServiceBrowser *sb = NULL;
2015
2123
  error_t ret_errno;
2204
2312
      goto end;
2205
2313
    }
2206
2314
  }
2207
 
    
 
2315
  
2208
2316
  {
2209
2317
    /* Work around Debian bug #633582:
2210
2318
       <http://bugs.debian.org/633582> */
2237
2345
          TEMP_FAILURE_RETRY(close(seckey_fd));
2238
2346
        }
2239
2347
      }
2240
 
    
 
2348
      
2241
2349
      if(strcmp(pubkey, PATHDIR "/" PUBKEY) == 0){
2242
2350
        int pubkey_fd = open(pubkey, O_RDONLY);
2243
2351
        if(pubkey_fd == -1){
2258
2366
          TEMP_FAILURE_RETRY(close(pubkey_fd));
2259
2367
        }
2260
2368
      }
2261
 
    
 
2369
      
2262
2370
      /* Lower privileges */
2263
2371
      ret_errno = lower_privileges();
2264
2372
      if(ret_errno != 0){
2733
2841
  /* Removes the GPGME temp directory and all files inside */
2734
2842
  if(tempdir != NULL){
2735
2843
    struct dirent **direntries = NULL;
2736
 
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY |
2737
 
                                                  O_NOFOLLOW));
 
2844
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
 
2845
                                                  | O_NOFOLLOW
 
2846
                                                  | O_DIRECTORY
 
2847
                                                  | O_PATH));
2738
2848
    if(tempdir_fd == -1){
2739
2849
      perror_plus("open");
2740
2850
    } else {