/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-02-28 20:30:21 UTC
  • Revision ID: teddy@recompile.se-20160228203021-4rvy27vm4mx4r575
Client: Remove an incorrect "nonnull" attribute

* plugins.d/mandos-client.c (init_gnutls_global): Remove "nonnull"
                           attribute from "dhparamsfilename" argument.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
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(), strncpy() */
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,
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() */
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
1640
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1649
1642
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1650
1643
  if(ret == -1){
1651
1644
    if(debug){
1652
 
      old_errno = errno;
 
1645
      ret_errno = errno;
1653
1646
      perror_plus("ioctl SIOCGIFFLAGS");
1654
 
      errno = old_errno;
 
1647
      errno = ret_errno;
1655
1648
    }
1656
1649
    return false;
1657
1650
  }
2083
2076
}
2084
2077
 
2085
2078
__attribute__((nonnull, warn_unused_result))
2086
 
int bring_up_interface(const char *const interface,
2087
 
                       const float delay){
2088
 
  int old_errno = errno;
 
2079
error_t bring_up_interface(const char *const interface,
 
2080
                           const float delay){
 
2081
  error_t old_errno = errno;
2089
2082
  int ret;
2090
2083
  struct ifreq network;
2091
2084
  unsigned int if_index = if_nametoindex(interface);
2101
2094
  }
2102
2095
  
2103
2096
  if(not interface_is_up(interface)){
2104
 
    int ret_errno = 0;
2105
 
    int ioctl_errno = 0;
 
2097
    error_t ret_errno = 0, ioctl_errno = 0;
2106
2098
    if(not get_flags(interface, &network)){
2107
2099
      ret_errno = errno;
2108
2100
      fprintf_plus(stderr, "Failed to get flags for interface "
2211
2203
}
2212
2204
 
2213
2205
__attribute__((nonnull, warn_unused_result))
2214
 
int take_down_interface(const char *const interface){
2215
 
  int old_errno = errno;
 
2206
error_t take_down_interface(const char *const interface){
 
2207
  error_t old_errno = errno;
2216
2208
  struct ifreq network;
2217
2209
  unsigned int if_index = if_nametoindex(interface);
2218
2210
  if(if_index == 0){
2221
2213
    return ENXIO;
2222
2214
  }
2223
2215
  if(interface_is_up(interface)){
2224
 
    int ret_errno = 0;
2225
 
    int ioctl_errno = 0;
 
2216
    error_t ret_errno = 0, ioctl_errno = 0;
2226
2217
    if(not get_flags(interface, &network) and debug){
2227
2218
      ret_errno = errno;
2228
2219
      fprintf_plus(stderr, "Failed to get flags for interface "
2478
2469
                         .args_doc = "",
2479
2470
                         .doc = "Mandos client -- Get and decrypt"
2480
2471
                         " 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){
 
2472
    ret = argp_parse(&argp, argc, argv,
 
2473
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
2474
    switch(ret){
2484
2475
    case 0:
2485
2476
      break;
2486
2477
    case ENOMEM:
2487
2478
    default:
2488
 
      errno = ret_errno;
 
2479
      errno = ret;
2489
2480
      perror_plus("argp_parse");
2490
2481
      exitcode = EX_OSERR;
2491
2482
      goto end;
2497
2488
  
2498
2489
  {
2499
2490
    /* Work around Debian bug #633582:
2500
 
       <https://bugs.debian.org/633582> */
 
2491
       <http://bugs.debian.org/633582> */
2501
2492
    
2502
2493
    /* Re-raise privileges */
2503
 
    ret = raise_privileges();
2504
 
    if(ret != 0){
2505
 
      errno = ret;
 
2494
    ret_errno = raise_privileges();
 
2495
    if(ret_errno != 0){
 
2496
      errno = ret_errno;
2506
2497
      perror_plus("Failed to raise privileges");
2507
2498
    } else {
2508
2499
      struct stat st;
2572
2563
      }
2573
2564
      
2574
2565
      /* Lower privileges */
2575
 
      ret = lower_privileges();
2576
 
      if(ret != 0){
2577
 
        errno = ret;
 
2566
      ret_errno = lower_privileges();
 
2567
      if(ret_errno != 0){
 
2568
        errno = ret_errno;
2578
2569
        perror_plus("Failed to lower privileges");
2579
2570
      }
2580
2571
    }
2908
2899
    
2909
2900
    /* Allocate a new server */
2910
2901
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
2911
 
                                 &config, NULL, NULL, &ret);
 
2902
                                 &config, NULL, NULL, &ret_errno);
2912
2903
    
2913
2904
    /* Free the Avahi configuration data */
2914
2905
    avahi_server_config_free(&config);
2917
2908
  /* Check if creating the Avahi server object succeeded */
2918
2909
  if(mc.server == NULL){
2919
2910
    fprintf_plus(stderr, "Failed to create Avahi server: %s\n",
2920
 
                 avahi_strerror(ret));
 
2911
                 avahi_strerror(ret_errno));
2921
2912
    exitcode = EX_UNAVAILABLE;
2922
2913
    goto end;
2923
2914
  }
2958
2949
 end:
2959
2950
  
2960
2951
  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
 
    }
 
2952
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
2968
2953
  }
2969
2954
  
2970
2955
  /* Cleanup things */
2981
2966
  
2982
2967
  if(gnutls_initialized){
2983
2968
    gnutls_certificate_free_credentials(mc.cred);
 
2969
    gnutls_global_deinit();
2984
2970
    gnutls_dh_params_deinit(mc.dh_params);
2985
2971
  }
2986
2972
  
3009
2995
  
3010
2996
  /* Re-raise privileges */
3011
2997
  {
3012
 
    ret = raise_privileges();
3013
 
    if(ret != 0){
3014
 
      errno = ret;
 
2998
    ret_errno = raise_privileges();
 
2999
    if(ret_errno != 0){
 
3000
      errno = ret_errno;
3015
3001
      perror_plus("Failed to raise privileges");
3016
3002
    } else {
3017
3003
      
3025
3011
        while((interface=argz_next(interfaces_to_take_down,
3026
3012
                                   interfaces_to_take_down_size,
3027
3013
                                   interface))){
3028
 
          ret = take_down_interface(interface);
3029
 
          if(ret != 0){
3030
 
            errno = ret;
 
3014
          ret_errno = take_down_interface(interface);
 
3015
          if(ret_errno != 0){
 
3016
            errno = ret_errno;
3031
3017
            perror_plus("Failed to take down interface");
3032
3018
          }
3033
3019
        }
3038
3024
      }
3039
3025
    }
3040
3026
    
3041
 
    ret = lower_privileges_permanently();
3042
 
    if(ret != 0){
3043
 
      errno = ret;
 
3027
    ret_errno = lower_privileges_permanently();
 
3028
    if(ret_errno != 0){
 
3029
      errno = ret_errno;
3044
3030
      perror_plus("Failed to lower privileges permanently");
3045
3031
    }
3046
3032
  }
3048
3034
  free(interfaces_to_take_down);
3049
3035
  free(interfaces_hooks);
3050
3036
  
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
3037
  /* Removes the GPGME temp directory and all files inside */
3108
3038
  if(tempdir != NULL){
3109
 
    clean_dir_at(-1, tempdir, 0);
 
3039
    struct dirent **direntries = NULL;
 
3040
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
 
3041
                                                  | O_NOFOLLOW
 
3042
                                                  | O_DIRECTORY
 
3043
                                                  | O_PATH));
 
3044
    if(tempdir_fd == -1){
 
3045
      perror_plus("open");
 
3046
    } else {
 
3047
      int numentries = scandirat(tempdir_fd, ".", &direntries,
 
3048
                                 notdotentries, alphasort);
 
3049
      if(numentries >= 0){
 
3050
        for(int i = 0; i < numentries; i++){
 
3051
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
 
3052
          if(ret == -1){
 
3053
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
 
3054
                         " \"%s\", 0): %s\n", tempdir,
 
3055
                         direntries[i]->d_name, strerror(errno));
 
3056
          }
 
3057
          free(direntries[i]);
 
3058
        }
 
3059
        
 
3060
        /* need to clean even if 0 because man page doesn't specify */
 
3061
        free(direntries);
 
3062
        if(numentries == -1){
 
3063
          perror_plus("scandir");
 
3064
        }
 
3065
        ret = rmdir(tempdir);
 
3066
        if(ret == -1 and errno != ENOENT){
 
3067
          perror_plus("rmdir");
 
3068
        }
 
3069
      }
 
3070
      close(tempdir_fd);
 
3071
    }
3110
3072
  }
3111
3073
  
3112
3074
  if(quit_now){