/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-20 03:03:33 UTC
  • Revision ID: teddy@recompile.se-20150720030333-203m2aeblypcsfte
Bug fix for GnuTLS 3: be compatible with old 2048-bit DSA keys.

The mandos-keygen program in Mandos version 1.6.0 and older generated
2048-bit DSA keys, and when GnuTLS uses these it has trouble
connecting using the Mandos default priority string.  This was
previously fixed in Mandos 1.6.2, but the bug reappeared when using
GnuTLS 3, so the default priority string has to change again; this
time also the Mandos client has to change its default, so now the
server and the client should use the same default priority string:

SECURE256:!CTYPE-X.509:+CTYPE-OPENPGP:!RSA:+SIGN-DSA-SHA256

* mandos (main/server_defaults): Changed default priority string.
* mandos-options.xml (/section/para[id="priority_compat"]): Removed.
  (/section/para[id="priority"]): Changed default priority string.
* mandos.conf ([DEFAULT]/priority): - '' -
* mandos.conf.xml (OPTIONS/priority): Refer to the id "priority"
                                      instead of "priority_compat".
* mandos.xml (OPTIONS/--priority): - '' -
* plugins.d/mandos-client.c (main): Changed default priority string.

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-2016 Teddy Hogeborn
13
 
 * Copyright © 2008-2016 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
47
47
                                   strtof(), abort() */
48
48
#include <stdbool.h>            /* bool, false, true */
49
49
#include <string.h>             /* strcmp(), strlen(), strerror(),
50
 
                                   asprintf(), strncpy(), strsignal()
51
 
                                */
 
50
                                   asprintf(), strcpy() */
52
51
#include <sys/ioctl.h>          /* ioctl */
53
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
54
53
                                   sockaddr_in6, PF_INET6,
58
57
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
59
58
                                   inet_pton(), connect(),
60
59
                                   getnameinfo() */
61
 
#include <fcntl.h>              /* open(), unlinkat(), AT_REMOVEDIR */
 
60
#include <fcntl.h>              /* open(), unlinkat() */
62
61
#include <dirent.h>             /* opendir(), struct dirent, readdir()
63
62
                                 */
64
63
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
65
64
                                   strtoimax() */
66
 
#include <errno.h>              /* perror(), errno, EINTR, EINVAL,
67
 
                                   EAI_SYSTEM, ENETUNREACH,
68
 
                                   EHOSTUNREACH, ECONNREFUSED, EPROTO,
69
 
                                   EIO, ENOENT, ENXIO, ENOMEM, EISDIR,
70
 
                                   ENOTEMPTY,
 
65
#include <errno.h>              /* perror(), errno,
71
66
                                   program_invocation_short_name */
72
67
#include <time.h>               /* nanosleep(), time(), sleep() */
73
68
#include <net/if.h>             /* ioctl, ifreq, SIOCGIFFLAGS, IFF_UP,
518
513
  fprintf_plus(stderr, "GnuTLS: %s", string);
519
514
}
520
515
 
521
 
__attribute__((nonnull(1, 2, 4), warn_unused_result))
 
516
__attribute__((nonnull, warn_unused_result))
522
517
static int init_gnutls_global(const char *pubkeyfilename,
523
518
                              const char *seckeyfilename,
524
519
                              const char *dhparamsfilename,
530
525
    fprintf_plus(stderr, "Initializing GnuTLS\n");
531
526
  }
532
527
  
 
528
  ret = gnutls_global_init();
 
529
  if(ret != GNUTLS_E_SUCCESS){
 
530
    fprintf_plus(stderr, "GnuTLS global_init: %s\n",
 
531
                 safer_gnutls_strerror(ret));
 
532
    return -1;
 
533
  }
 
534
  
533
535
  if(debug){
534
536
    /* "Use a log level over 10 to enable all debugging options."
535
537
     * - GnuTLS manual
543
545
  if(ret != GNUTLS_E_SUCCESS){
544
546
    fprintf_plus(stderr, "GnuTLS memory error: %s\n",
545
547
                 safer_gnutls_strerror(ret));
 
548
    gnutls_global_deinit();
546
549
    return -1;
547
550
  }
548
551
  
752
755
 globalfail:
753
756
  
754
757
  gnutls_certificate_free_credentials(mc->cred);
 
758
  gnutls_global_deinit();
755
759
  gnutls_dh_params_deinit(mc->dh_params);
756
760
  return -1;
757
761
}
818
822
 
819
823
/* Set effective uid to 0, return errno */
820
824
__attribute__((warn_unused_result))
821
 
int raise_privileges(void){
822
 
  int old_errno = errno;
823
 
  int ret = 0;
 
825
error_t raise_privileges(void){
 
826
  error_t old_errno = errno;
 
827
  error_t ret_errno = 0;
824
828
  if(seteuid(0) == -1){
825
 
    ret = errno;
 
829
    ret_errno = errno;
826
830
  }
827
831
  errno = old_errno;
828
 
  return ret;
 
832
  return ret_errno;
829
833
}
830
834
 
831
835
/* Set effective and real user ID to 0.  Return errno. */
832
836
__attribute__((warn_unused_result))
833
 
int raise_privileges_permanently(void){
834
 
  int old_errno = errno;
835
 
  int ret = raise_privileges();
836
 
  if(ret != 0){
 
837
error_t raise_privileges_permanently(void){
 
838
  error_t old_errno = errno;
 
839
  error_t ret_errno = raise_privileges();
 
840
  if(ret_errno != 0){
837
841
    errno = old_errno;
838
 
    return ret;
 
842
    return ret_errno;
839
843
  }
840
844
  if(setuid(0) == -1){
841
 
    ret = errno;
 
845
    ret_errno = errno;
842
846
  }
843
847
  errno = old_errno;
844
 
  return ret;
 
848
  return ret_errno;
845
849
}
846
850
 
847
851
/* Set effective user ID to unprivileged saved user ID */
848
852
__attribute__((warn_unused_result))
849
 
int lower_privileges(void){
850
 
  int old_errno = errno;
851
 
  int ret = 0;
 
853
error_t lower_privileges(void){
 
854
  error_t old_errno = errno;
 
855
  error_t ret_errno = 0;
852
856
  if(seteuid(uid) == -1){
853
 
    ret = errno;
 
857
    ret_errno = errno;
854
858
  }
855
859
  errno = old_errno;
856
 
  return ret;
 
860
  return ret_errno;
857
861
}
858
862
 
859
863
/* Lower privileges permanently */
860
864
__attribute__((warn_unused_result))
861
 
int lower_privileges_permanently(void){
862
 
  int old_errno = errno;
863
 
  int ret = 0;
 
865
error_t lower_privileges_permanently(void){
 
866
  error_t old_errno = errno;
 
867
  error_t ret_errno = 0;
864
868
  if(setuid(uid) == -1){
865
 
    ret = errno;
 
869
    ret_errno = errno;
866
870
  }
867
871
  errno = old_errno;
868
 
  return ret;
 
872
  return ret_errno;
869
873
}
870
874
 
871
875
/* Helper function to add_local_route() and delete_local_route() */
1219
1223
                    sizeof(struct sockaddr_in));
1220
1224
    }
1221
1225
    if(ret < 0){
1222
 
      if(((errno == ENETUNREACH) or (errno == EHOSTUNREACH))
 
1226
      if(errno == ENETUNREACH
1223
1227
         and if_index != AVAHI_IF_UNSPEC
1224
1228
         and connect_to == NULL
1225
1229
         and not route_added and
1238
1242
           with an explicit route added with the server's address.
1239
1243
           
1240
1244
           Avahi bug reference:
1241
 
           https://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1245
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
1242
1246
           https://bugs.debian.org/587961
1243
1247
        */
1244
1248
        if(debug){
1424
1428
                                               &decrypted_buffer, mc);
1425
1429
    if(decrypted_buffer_size >= 0){
1426
1430
      
1427
 
      clearerr(stdout);
1428
1431
      written = 0;
1429
1432
      while(written < (size_t) decrypted_buffer_size){
1430
1433
        if(quit_now){
1446
1449
        }
1447
1450
        written += (size_t)ret;
1448
1451
      }
1449
 
      ret = fflush(stdout);
1450
 
      if(ret != 0){
1451
 
        int e = errno;
1452
 
        if(debug){
1453
 
          fprintf_plus(stderr, "Error writing encrypted data: %s\n",
1454
 
                       strerror(errno));
1455
 
        }
1456
 
        errno = e;
1457
 
        goto mandos_end;
1458
 
      }
1459
1452
      retval = 0;
1460
1453
    }
1461
1454
  }
1635
1628
__attribute__((nonnull, warn_unused_result))
1636
1629
bool get_flags(const char *ifname, struct ifreq *ifr){
1637
1630
  int ret;
1638
 
  int old_errno;
 
1631
  error_t ret_errno;
1639
1632
  
1640
1633
  int s = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
1641
1634
  if(s < 0){
1642
 
    old_errno = errno;
 
1635
    ret_errno = errno;
1643
1636
    perror_plus("socket");
1644
 
    errno = old_errno;
 
1637
    errno = ret_errno;
1645
1638
    return false;
1646
1639
  }
1647
 
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1648
 
  ifr->ifr_name[IF_NAMESIZE-1] = '\0'; /* NUL terminate */
 
1640
  strcpy(ifr->ifr_name, ifname);
1649
1641
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1650
1642
  if(ret == -1){
1651
1643
    if(debug){
1652
 
      old_errno = errno;
 
1644
      ret_errno = errno;
1653
1645
      perror_plus("ioctl SIOCGIFFLAGS");
1654
 
      errno = old_errno;
 
1646
      errno = ret_errno;
1655
1647
    }
1656
1648
    return false;
1657
1649
  }
1921
1913
      return;
1922
1914
    }
1923
1915
  }
 
1916
#ifdef __GLIBC__
 
1917
#if __GLIBC_PREREQ(2, 15)
1924
1918
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1925
1919
                           runnable_hook, alphasort);
 
1920
#else  /* not __GLIBC_PREREQ(2, 15) */
 
1921
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1922
                         alphasort);
 
1923
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
1924
#else   /* not __GLIBC__ */
 
1925
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1926
                         alphasort);
 
1927
#endif  /* not __GLIBC__ */
1926
1928
  if(numhooks == -1){
1927
1929
    perror_plus("scandir");
1928
1930
    return;
2083
2085
}
2084
2086
 
2085
2087
__attribute__((nonnull, warn_unused_result))
2086
 
int bring_up_interface(const char *const interface,
2087
 
                       const float delay){
2088
 
  int old_errno = errno;
 
2088
error_t bring_up_interface(const char *const interface,
 
2089
                           const float delay){
 
2090
  error_t old_errno = errno;
2089
2091
  int ret;
2090
2092
  struct ifreq network;
2091
2093
  unsigned int if_index = if_nametoindex(interface);
2101
2103
  }
2102
2104
  
2103
2105
  if(not interface_is_up(interface)){
2104
 
    int ret_errno = 0;
2105
 
    int ioctl_errno = 0;
 
2106
    error_t ret_errno = 0, ioctl_errno = 0;
2106
2107
    if(not get_flags(interface, &network)){
2107
2108
      ret_errno = errno;
2108
2109
      fprintf_plus(stderr, "Failed to get flags for interface "
2211
2212
}
2212
2213
 
2213
2214
__attribute__((nonnull, warn_unused_result))
2214
 
int take_down_interface(const char *const interface){
2215
 
  int old_errno = errno;
 
2215
error_t take_down_interface(const char *const interface){
 
2216
  error_t old_errno = errno;
2216
2217
  struct ifreq network;
2217
2218
  unsigned int if_index = if_nametoindex(interface);
2218
2219
  if(if_index == 0){
2221
2222
    return ENXIO;
2222
2223
  }
2223
2224
  if(interface_is_up(interface)){
2224
 
    int ret_errno = 0;
2225
 
    int ioctl_errno = 0;
 
2225
    error_t ret_errno = 0, ioctl_errno = 0;
2226
2226
    if(not get_flags(interface, &network) and debug){
2227
2227
      ret_errno = errno;
2228
2228
      fprintf_plus(stderr, "Failed to get flags for interface "
2478
2478
                         .args_doc = "",
2479
2479
                         .doc = "Mandos client -- Get and decrypt"
2480
2480
                         " passwords from a Mandos server" };
2481
 
    ret_errno = argp_parse(&argp, argc, argv,
2482
 
                           ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
2483
 
    switch(ret_errno){
 
2481
    ret = argp_parse(&argp, argc, argv,
 
2482
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
2483
    switch(ret){
2484
2484
    case 0:
2485
2485
      break;
2486
2486
    case ENOMEM:
2487
2487
    default:
2488
 
      errno = ret_errno;
 
2488
      errno = ret;
2489
2489
      perror_plus("argp_parse");
2490
2490
      exitcode = EX_OSERR;
2491
2491
      goto end;
2497
2497
  
2498
2498
  {
2499
2499
    /* Work around Debian bug #633582:
2500
 
       <https://bugs.debian.org/633582> */
 
2500
       <http://bugs.debian.org/633582> */
2501
2501
    
2502
2502
    /* Re-raise privileges */
2503
 
    ret = raise_privileges();
2504
 
    if(ret != 0){
2505
 
      errno = ret;
 
2503
    ret_errno = raise_privileges();
 
2504
    if(ret_errno != 0){
 
2505
      errno = ret_errno;
2506
2506
      perror_plus("Failed to raise privileges");
2507
2507
    } else {
2508
2508
      struct stat st;
2572
2572
      }
2573
2573
      
2574
2574
      /* Lower privileges */
2575
 
      ret = lower_privileges();
2576
 
      if(ret != 0){
2577
 
        errno = ret;
 
2575
      ret_errno = lower_privileges();
 
2576
      if(ret_errno != 0){
 
2577
        errno = ret_errno;
2578
2578
        perror_plus("Failed to lower privileges");
2579
2579
      }
2580
2580
    }
2908
2908
    
2909
2909
    /* Allocate a new server */
2910
2910
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
2911
 
                                 &config, NULL, NULL, &ret);
 
2911
                                 &config, NULL, NULL, &ret_errno);
2912
2912
    
2913
2913
    /* Free the Avahi configuration data */
2914
2914
    avahi_server_config_free(&config);
2917
2917
  /* Check if creating the Avahi server object succeeded */
2918
2918
  if(mc.server == NULL){
2919
2919
    fprintf_plus(stderr, "Failed to create Avahi server: %s\n",
2920
 
                 avahi_strerror(ret));
 
2920
                 avahi_strerror(ret_errno));
2921
2921
    exitcode = EX_UNAVAILABLE;
2922
2922
    goto end;
2923
2923
  }
2958
2958
 end:
2959
2959
  
2960
2960
  if(debug){
2961
 
    if(signal_received){
2962
 
      fprintf_plus(stderr, "%s exiting due to signal %d: %s\n",
2963
 
                   argv[0], signal_received,
2964
 
                   strsignal(signal_received));
2965
 
    } else {
2966
 
      fprintf_plus(stderr, "%s exiting\n", argv[0]);
2967
 
    }
 
2961
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
2968
2962
  }
2969
2963
  
2970
2964
  /* Cleanup things */
2981
2975
  
2982
2976
  if(gnutls_initialized){
2983
2977
    gnutls_certificate_free_credentials(mc.cred);
 
2978
    gnutls_global_deinit();
2984
2979
    gnutls_dh_params_deinit(mc.dh_params);
2985
2980
  }
2986
2981
  
3009
3004
  
3010
3005
  /* Re-raise privileges */
3011
3006
  {
3012
 
    ret = raise_privileges();
3013
 
    if(ret != 0){
3014
 
      errno = ret;
 
3007
    ret_errno = raise_privileges();
 
3008
    if(ret_errno != 0){
 
3009
      errno = ret_errno;
3015
3010
      perror_plus("Failed to raise privileges");
3016
3011
    } else {
3017
3012
      
3025
3020
        while((interface=argz_next(interfaces_to_take_down,
3026
3021
                                   interfaces_to_take_down_size,
3027
3022
                                   interface))){
3028
 
          ret = take_down_interface(interface);
3029
 
          if(ret != 0){
3030
 
            errno = ret;
 
3023
          ret_errno = take_down_interface(interface);
 
3024
          if(ret_errno != 0){
 
3025
            errno = ret_errno;
3031
3026
            perror_plus("Failed to take down interface");
3032
3027
          }
3033
3028
        }
3038
3033
      }
3039
3034
    }
3040
3035
    
3041
 
    ret = lower_privileges_permanently();
3042
 
    if(ret != 0){
3043
 
      errno = ret;
 
3036
    ret_errno = lower_privileges_permanently();
 
3037
    if(ret_errno != 0){
 
3038
      errno = ret_errno;
3044
3039
      perror_plus("Failed to lower privileges permanently");
3045
3040
    }
3046
3041
  }
3048
3043
  free(interfaces_to_take_down);
3049
3044
  free(interfaces_hooks);
3050
3045
  
3051
 
  void clean_dir_at(int base, const char * const dirname,
3052
 
                    uintmax_t level){
3053
 
    struct dirent **direntries = NULL;
3054
 
    int dret;
3055
 
    int dir_fd = (int)TEMP_FAILURE_RETRY(openat(base, dirname,
3056
 
                                                O_RDONLY
3057
 
                                                | O_NOFOLLOW
3058
 
                                                | O_DIRECTORY
3059
 
                                                | O_PATH));
3060
 
    if(dir_fd == -1){
3061
 
      perror_plus("open");
3062
 
    }
3063
 
    int numentries = scandirat(dir_fd, ".", &direntries,
3064
 
                               notdotentries, alphasort);
3065
 
    if(numentries >= 0){
3066
 
      for(int i = 0; i < numentries; i++){
3067
 
        if(debug){
3068
 
          fprintf_plus(stderr, "Unlinking \"%s/%s\"\n",
3069
 
                       dirname, direntries[i]->d_name);
3070
 
        }
3071
 
        dret = unlinkat(dir_fd, direntries[i]->d_name, 0);
3072
 
        if(dret == -1){
3073
 
          if(errno == EISDIR){
3074
 
              dret = unlinkat(dir_fd, direntries[i]->d_name,
3075
 
                              AT_REMOVEDIR);
3076
 
          }         
3077
 
          if((dret == -1) and (errno == ENOTEMPTY)
3078
 
             and (strcmp(direntries[i]->d_name, "private-keys-v1.d")
3079
 
                  == 0) and (level == 0)){
3080
 
            /* Recurse only in this special case */
3081
 
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3082
 
            dret = 0;
3083
 
          }
3084
 
          if(dret == -1){
3085
 
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3086
 
                         direntries[i]->d_name, strerror(errno));
3087
 
          }
3088
 
        }
3089
 
        free(direntries[i]);
3090
 
      }
3091
 
      
3092
 
      /* need to clean even if 0 because man page doesn't specify */
3093
 
      free(direntries);
3094
 
      if(numentries == -1){
3095
 
        perror_plus("scandirat");
3096
 
      }
3097
 
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3098
 
      if(dret == -1 and errno != ENOENT){
3099
 
        perror_plus("rmdir");
3100
 
      }
3101
 
    } else {
3102
 
      perror_plus("scandirat");
3103
 
    }
3104
 
    close(dir_fd);
3105
 
  }
3106
 
  
3107
3046
  /* Removes the GPGME temp directory and all files inside */
3108
3047
  if(tempdir != NULL){
3109
 
    clean_dir_at(-1, tempdir, 0);
 
3048
    struct dirent **direntries = NULL;
 
3049
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
 
3050
                                                  | O_NOFOLLOW
 
3051
                                                  | O_DIRECTORY
 
3052
                                                  | O_PATH));
 
3053
    if(tempdir_fd == -1){
 
3054
      perror_plus("open");
 
3055
    } else {
 
3056
#ifdef __GLIBC__
 
3057
#if __GLIBC_PREREQ(2, 15)
 
3058
      int numentries = scandirat(tempdir_fd, ".", &direntries,
 
3059
                                 notdotentries, alphasort);
 
3060
#else  /* not __GLIBC_PREREQ(2, 15) */
 
3061
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
3062
                               alphasort);
 
3063
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
3064
#else   /* not __GLIBC__ */
 
3065
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
3066
                               alphasort);
 
3067
#endif  /* not __GLIBC__ */
 
3068
      if(numentries >= 0){
 
3069
        for(int i = 0; i < numentries; i++){
 
3070
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
 
3071
          if(ret == -1){
 
3072
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
 
3073
                         " \"%s\", 0): %s\n", tempdir,
 
3074
                         direntries[i]->d_name, strerror(errno));
 
3075
          }
 
3076
          free(direntries[i]);
 
3077
        }
 
3078
        
 
3079
        /* need to clean even if 0 because man page doesn't specify */
 
3080
        free(direntries);
 
3081
        if(numentries == -1){
 
3082
          perror_plus("scandir");
 
3083
        }
 
3084
        ret = rmdir(tempdir);
 
3085
        if(ret == -1 and errno != ENOENT){
 
3086
          perror_plus("rmdir");
 
3087
        }
 
3088
      }
 
3089
      close(tempdir_fd);
 
3090
    }
3110
3091
  }
3111
3092
  
3112
3093
  if(quit_now){