/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: 2018-02-19 21:32:07 UTC
  • Revision ID: teddy@recompile.se-20180219213207-0un0ylegx390pftq
Client bug fixes: Fix file descriptor leaks

* plugin-helpers/mandos-client.c (init_gnutls_global, get_flags):
  Always close files and sockets after they are used.

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
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
 
12
 * Copyright © 2008-2018 Teddy Hogeborn
 
13
 * Copyright © 2008-2018 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
21
23
 * WITHOUT ANY WARRANTY; without even the implied warranty of
22
24
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
23
25
 * General Public License for more details.
24
26
 * 
25
27
 * You should have received a copy of the GNU General Public License
26
 
 * along with this program.  If not, see
27
 
 * <http://www.gnu.org/licenses/>.
 
28
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
28
29
 * 
29
30
 * Contact the authors at <mandos@recompile.se>.
30
31
 */
47
48
                                   strtof(), abort() */
48
49
#include <stdbool.h>            /* bool, false, true */
49
50
#include <string.h>             /* strcmp(), strlen(), strerror(),
50
 
                                   asprintf(), strncpy() */
 
51
                                   asprintf(), strncpy(), strsignal()
 
52
                                */
51
53
#include <sys/ioctl.h>          /* ioctl */
52
54
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
53
55
                                   sockaddr_in6, PF_INET6,
57
59
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
58
60
                                   inet_pton(), connect(),
59
61
                                   getnameinfo() */
60
 
#include <fcntl.h>              /* open(), unlinkat() */
 
62
#include <fcntl.h>              /* open(), unlinkat(), AT_REMOVEDIR */
61
63
#include <dirent.h>             /* opendir(), struct dirent, readdir()
62
64
                                 */
63
65
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
64
66
                                   strtoimax() */
65
 
#include <errno.h>              /* perror(), errno,
 
67
#include <errno.h>              /* perror(), errno, EINTR, EINVAL,
 
68
                                   EAI_SYSTEM, ENETUNREACH,
 
69
                                   EHOSTUNREACH, ECONNREFUSED, EPROTO,
 
70
                                   EIO, ENOENT, ENXIO, ENOMEM, EISDIR,
 
71
                                   ENOTEMPTY,
66
72
                                   program_invocation_short_name */
67
73
#include <time.h>               /* nanosleep(), time(), sleep() */
68
74
#include <net/if.h>             /* ioctl, ifreq, SIOCGIFFLAGS, IFF_UP,
607
613
        }
608
614
        params.size += (unsigned int)bytes_read;
609
615
      }
 
616
      close(dhpfile);
610
617
      if(params.data == NULL){
611
618
        dhparamsfilename = NULL;
612
619
      }
621
628
                     safer_gnutls_strerror(ret));
622
629
        dhparamsfilename = NULL;
623
630
      }
 
631
      free(params.data);
624
632
    } while(false);
625
633
  }
626
634
  if(dhparamsfilename == NULL){
813
821
 
814
822
/* Set effective uid to 0, return errno */
815
823
__attribute__((warn_unused_result))
816
 
error_t raise_privileges(void){
817
 
  error_t old_errno = errno;
818
 
  error_t ret_errno = 0;
 
824
int raise_privileges(void){
 
825
  int old_errno = errno;
 
826
  int ret = 0;
819
827
  if(seteuid(0) == -1){
820
 
    ret_errno = errno;
 
828
    ret = errno;
821
829
  }
822
830
  errno = old_errno;
823
 
  return ret_errno;
 
831
  return ret;
824
832
}
825
833
 
826
834
/* Set effective and real user ID to 0.  Return errno. */
827
835
__attribute__((warn_unused_result))
828
 
error_t raise_privileges_permanently(void){
829
 
  error_t old_errno = errno;
830
 
  error_t ret_errno = raise_privileges();
831
 
  if(ret_errno != 0){
 
836
int raise_privileges_permanently(void){
 
837
  int old_errno = errno;
 
838
  int ret = raise_privileges();
 
839
  if(ret != 0){
832
840
    errno = old_errno;
833
 
    return ret_errno;
 
841
    return ret;
834
842
  }
835
843
  if(setuid(0) == -1){
836
 
    ret_errno = errno;
 
844
    ret = errno;
837
845
  }
838
846
  errno = old_errno;
839
 
  return ret_errno;
 
847
  return ret;
840
848
}
841
849
 
842
850
/* Set effective user ID to unprivileged saved user ID */
843
851
__attribute__((warn_unused_result))
844
 
error_t lower_privileges(void){
845
 
  error_t old_errno = errno;
846
 
  error_t ret_errno = 0;
 
852
int lower_privileges(void){
 
853
  int old_errno = errno;
 
854
  int ret = 0;
847
855
  if(seteuid(uid) == -1){
848
 
    ret_errno = errno;
 
856
    ret = errno;
849
857
  }
850
858
  errno = old_errno;
851
 
  return ret_errno;
 
859
  return ret;
852
860
}
853
861
 
854
862
/* Lower privileges permanently */
855
863
__attribute__((warn_unused_result))
856
 
error_t lower_privileges_permanently(void){
857
 
  error_t old_errno = errno;
858
 
  error_t ret_errno = 0;
 
864
int lower_privileges_permanently(void){
 
865
  int old_errno = errno;
 
866
  int ret = 0;
859
867
  if(setuid(uid) == -1){
860
 
    ret_errno = errno;
 
868
    ret = errno;
861
869
  }
862
870
  errno = old_errno;
863
 
  return ret_errno;
 
871
  return ret;
864
872
}
865
873
 
866
874
/* Helper function to add_local_route() and delete_local_route() */
1073
1081
    bool match = false;
1074
1082
    {
1075
1083
      char *interface = NULL;
1076
 
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
1077
 
                                 interface))){
 
1084
      while((interface = argz_next(mc->interfaces,
 
1085
                                   mc->interfaces_size,
 
1086
                                   interface))){
1078
1087
        if(if_nametoindex(interface) == (unsigned int)if_index){
1079
1088
          match = true;
1080
1089
          break;
1233
1242
           with an explicit route added with the server's address.
1234
1243
           
1235
1244
           Avahi bug reference:
1236
 
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1245
           https://lists.freedesktop.org/archives/avahi/2010-February/001833.html
1237
1246
           https://bugs.debian.org/587961
1238
1247
        */
1239
1248
        if(debug){
1419
1428
                                               &decrypted_buffer, mc);
1420
1429
    if(decrypted_buffer_size >= 0){
1421
1430
      
 
1431
      clearerr(stdout);
1422
1432
      written = 0;
1423
1433
      while(written < (size_t) decrypted_buffer_size){
1424
1434
        if(quit_now){
1440
1450
        }
1441
1451
        written += (size_t)ret;
1442
1452
      }
 
1453
      ret = fflush(stdout);
 
1454
      if(ret != 0){
 
1455
        int e = errno;
 
1456
        if(debug){
 
1457
          fprintf_plus(stderr, "Error writing encrypted data: %s\n",
 
1458
                       strerror(errno));
 
1459
        }
 
1460
        errno = e;
 
1461
        goto mandos_end;
 
1462
      }
1443
1463
      retval = 0;
1444
1464
    }
1445
1465
  }
1476
1496
  return retval;
1477
1497
}
1478
1498
 
1479
 
__attribute__((nonnull))
1480
1499
static void resolve_callback(AvahiSServiceResolver *r,
1481
1500
                             AvahiIfIndex interface,
1482
1501
                             AvahiProtocol proto,
1619
1638
__attribute__((nonnull, warn_unused_result))
1620
1639
bool get_flags(const char *ifname, struct ifreq *ifr){
1621
1640
  int ret;
1622
 
  error_t ret_errno;
 
1641
  int old_errno;
1623
1642
  
1624
1643
  int s = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
1625
1644
  if(s < 0){
1626
 
    ret_errno = errno;
 
1645
    old_errno = errno;
1627
1646
    perror_plus("socket");
1628
 
    errno = ret_errno;
 
1647
    errno = old_errno;
1629
1648
    return false;
1630
1649
  }
1631
1650
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1633
1652
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1634
1653
  if(ret == -1){
1635
1654
    if(debug){
1636
 
      ret_errno = errno;
 
1655
      old_errno = errno;
1637
1656
      perror_plus("ioctl SIOCGIFFLAGS");
1638
 
      errno = ret_errno;
 
1657
      errno = old_errno;
1639
1658
    }
 
1659
    close(s);
1640
1660
    return false;
1641
1661
  }
 
1662
  close(s);
1642
1663
  return true;
1643
1664
}
1644
1665
 
1905
1926
      return;
1906
1927
    }
1907
1928
  }
 
1929
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1930
  if(devnull == -1){
 
1931
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1932
    return;
 
1933
  }
1908
1934
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1909
1935
                           runnable_hook, alphasort);
1910
1936
  if(numhooks == -1){
1911
1937
    perror_plus("scandir");
 
1938
    close(devnull);
1912
1939
    return;
1913
1940
  }
1914
1941
  struct dirent *direntry;
1915
1942
  int ret;
1916
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1917
 
  if(devnull == -1){
1918
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1919
 
    return;
1920
 
  }
1921
1943
  for(int i = 0; i < numhooks; i++){
1922
1944
    direntry = direntries[i];
1923
1945
    if(debug){
2067
2089
}
2068
2090
 
2069
2091
__attribute__((nonnull, warn_unused_result))
2070
 
error_t bring_up_interface(const char *const interface,
2071
 
                           const float delay){
2072
 
  error_t old_errno = errno;
 
2092
int bring_up_interface(const char *const interface,
 
2093
                       const float delay){
 
2094
  int old_errno = errno;
2073
2095
  int ret;
2074
2096
  struct ifreq network;
2075
2097
  unsigned int if_index = if_nametoindex(interface);
2085
2107
  }
2086
2108
  
2087
2109
  if(not interface_is_up(interface)){
2088
 
    error_t ret_errno = 0, ioctl_errno = 0;
 
2110
    int ret_errno = 0;
 
2111
    int ioctl_errno = 0;
2089
2112
    if(not get_flags(interface, &network)){
2090
2113
      ret_errno = errno;
2091
2114
      fprintf_plus(stderr, "Failed to get flags for interface "
2178
2201
  
2179
2202
  /* Sleep checking until interface is running.
2180
2203
     Check every 0.25s, up to total time of delay */
2181
 
  for(int i=0; i < delay * 4; i++){
 
2204
  for(int i = 0; i < delay * 4; i++){
2182
2205
    if(interface_is_running(interface)){
2183
2206
      break;
2184
2207
    }
2194
2217
}
2195
2218
 
2196
2219
__attribute__((nonnull, warn_unused_result))
2197
 
error_t take_down_interface(const char *const interface){
2198
 
  error_t old_errno = errno;
 
2220
int take_down_interface(const char *const interface){
 
2221
  int old_errno = errno;
2199
2222
  struct ifreq network;
2200
2223
  unsigned int if_index = if_nametoindex(interface);
2201
2224
  if(if_index == 0){
2204
2227
    return ENXIO;
2205
2228
  }
2206
2229
  if(interface_is_up(interface)){
2207
 
    error_t ret_errno = 0, ioctl_errno = 0;
 
2230
    int ret_errno = 0;
 
2231
    int ioctl_errno = 0;
2208
2232
    if(not get_flags(interface, &network) and debug){
2209
2233
      ret_errno = errno;
2210
2234
      fprintf_plus(stderr, "Failed to get flags for interface "
2460
2484
                         .args_doc = "",
2461
2485
                         .doc = "Mandos client -- Get and decrypt"
2462
2486
                         " passwords from a Mandos server" };
2463
 
    ret = argp_parse(&argp, argc, argv,
2464
 
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
2465
 
    switch(ret){
 
2487
    ret_errno = argp_parse(&argp, argc, argv,
 
2488
                           ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
2489
    switch(ret_errno){
2466
2490
    case 0:
2467
2491
      break;
2468
2492
    case ENOMEM:
2469
2493
    default:
2470
 
      errno = ret;
 
2494
      errno = ret_errno;
2471
2495
      perror_plus("argp_parse");
2472
2496
      exitcode = EX_OSERR;
2473
2497
      goto end;
2479
2503
  
2480
2504
  {
2481
2505
    /* Work around Debian bug #633582:
2482
 
       <http://bugs.debian.org/633582> */
 
2506
       <https://bugs.debian.org/633582> */
2483
2507
    
2484
2508
    /* Re-raise privileges */
2485
 
    ret_errno = raise_privileges();
2486
 
    if(ret_errno != 0){
2487
 
      errno = ret_errno;
 
2509
    ret = raise_privileges();
 
2510
    if(ret != 0){
 
2511
      errno = ret;
2488
2512
      perror_plus("Failed to raise privileges");
2489
2513
    } else {
2490
2514
      struct stat st;
2554
2578
      }
2555
2579
      
2556
2580
      /* Lower privileges */
2557
 
      ret_errno = lower_privileges();
2558
 
      if(ret_errno != 0){
2559
 
        errno = ret_errno;
 
2581
      ret = lower_privileges();
 
2582
      if(ret != 0){
 
2583
        errno = ret;
2560
2584
        perror_plus("Failed to lower privileges");
2561
2585
      }
2562
2586
    }
2890
2914
    
2891
2915
    /* Allocate a new server */
2892
2916
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
2893
 
                                 &config, NULL, NULL, &ret_errno);
 
2917
                                 &config, NULL, NULL, &ret);
2894
2918
    
2895
2919
    /* Free the Avahi configuration data */
2896
2920
    avahi_server_config_free(&config);
2899
2923
  /* Check if creating the Avahi server object succeeded */
2900
2924
  if(mc.server == NULL){
2901
2925
    fprintf_plus(stderr, "Failed to create Avahi server: %s\n",
2902
 
                 avahi_strerror(ret_errno));
 
2926
                 avahi_strerror(ret));
2903
2927
    exitcode = EX_UNAVAILABLE;
2904
2928
    goto end;
2905
2929
  }
2940
2964
 end:
2941
2965
  
2942
2966
  if(debug){
2943
 
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
 
2967
    if(signal_received){
 
2968
      fprintf_plus(stderr, "%s exiting due to signal %d: %s\n",
 
2969
                   argv[0], signal_received,
 
2970
                   strsignal(signal_received));
 
2971
    } else {
 
2972
      fprintf_plus(stderr, "%s exiting\n", argv[0]);
 
2973
    }
2944
2974
  }
2945
2975
  
2946
2976
  /* Cleanup things */
2985
3015
  
2986
3016
  /* Re-raise privileges */
2987
3017
  {
2988
 
    ret_errno = raise_privileges();
2989
 
    if(ret_errno != 0){
2990
 
      errno = ret_errno;
 
3018
    ret = raise_privileges();
 
3019
    if(ret != 0){
 
3020
      errno = ret;
2991
3021
      perror_plus("Failed to raise privileges");
2992
3022
    } else {
2993
3023
      
2998
3028
      /* Take down the network interfaces which were brought up */
2999
3029
      {
3000
3030
        char *interface = NULL;
3001
 
        while((interface=argz_next(interfaces_to_take_down,
3002
 
                                   interfaces_to_take_down_size,
3003
 
                                   interface))){
3004
 
          ret_errno = take_down_interface(interface);
3005
 
          if(ret_errno != 0){
3006
 
            errno = ret_errno;
 
3031
        while((interface = argz_next(interfaces_to_take_down,
 
3032
                                     interfaces_to_take_down_size,
 
3033
                                     interface))){
 
3034
          ret = take_down_interface(interface);
 
3035
          if(ret != 0){
 
3036
            errno = ret;
3007
3037
            perror_plus("Failed to take down interface");
3008
3038
          }
3009
3039
        }
3014
3044
      }
3015
3045
    }
3016
3046
    
3017
 
    ret_errno = lower_privileges_permanently();
3018
 
    if(ret_errno != 0){
3019
 
      errno = ret_errno;
 
3047
    ret = lower_privileges_permanently();
 
3048
    if(ret != 0){
 
3049
      errno = ret;
3020
3050
      perror_plus("Failed to lower privileges permanently");
3021
3051
    }
3022
3052
  }
3024
3054
  free(interfaces_to_take_down);
3025
3055
  free(interfaces_hooks);
3026
3056
  
 
3057
  void clean_dir_at(int base, const char * const dirname,
 
3058
                    uintmax_t level){
 
3059
    struct dirent **direntries = NULL;
 
3060
    int dret;
 
3061
    int dir_fd = (int)TEMP_FAILURE_RETRY(openat(base, dirname,
 
3062
                                                O_RDONLY
 
3063
                                                | O_NOFOLLOW
 
3064
                                                | O_DIRECTORY
 
3065
                                                | O_PATH));
 
3066
    if(dir_fd == -1){
 
3067
      perror_plus("open");
 
3068
      return;
 
3069
    }
 
3070
    int numentries = scandirat(dir_fd, ".", &direntries,
 
3071
                               notdotentries, alphasort);
 
3072
    if(numentries >= 0){
 
3073
      for(int i = 0; i < numentries; i++){
 
3074
        if(debug){
 
3075
          fprintf_plus(stderr, "Unlinking \"%s/%s\"\n",
 
3076
                       dirname, direntries[i]->d_name);
 
3077
        }
 
3078
        dret = unlinkat(dir_fd, direntries[i]->d_name, 0);
 
3079
        if(dret == -1){
 
3080
          if(errno == EISDIR){
 
3081
              dret = unlinkat(dir_fd, direntries[i]->d_name,
 
3082
                              AT_REMOVEDIR);
 
3083
          }         
 
3084
          if((dret == -1) and (errno == ENOTEMPTY)
 
3085
             and (strcmp(direntries[i]->d_name, "private-keys-v1.d")
 
3086
                  == 0) and (level == 0)){
 
3087
            /* Recurse only in this special case */
 
3088
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
 
3089
            dret = 0;
 
3090
          }
 
3091
          if((dret == -1) and (errno != ENOENT)){
 
3092
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
 
3093
                         direntries[i]->d_name, strerror(errno));
 
3094
          }
 
3095
        }
 
3096
        free(direntries[i]);
 
3097
      }
 
3098
      
 
3099
      /* need to clean even if 0 because man page doesn't specify */
 
3100
      free(direntries);
 
3101
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
 
3102
      if(dret == -1 and errno != ENOENT){
 
3103
        perror_plus("rmdir");
 
3104
      }
 
3105
    } else {
 
3106
      perror_plus("scandirat");
 
3107
    }
 
3108
    close(dir_fd);
 
3109
  }
 
3110
  
3027
3111
  /* Removes the GPGME temp directory and all files inside */
3028
3112
  if(tempdir != NULL){
3029
 
    struct dirent **direntries = NULL;
3030
 
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
3031
 
                                                  | O_NOFOLLOW
3032
 
                                                  | O_DIRECTORY
3033
 
                                                  | O_PATH));
3034
 
    if(tempdir_fd == -1){
3035
 
      perror_plus("open");
3036
 
    } else {
3037
 
      int numentries = scandirat(tempdir_fd, ".", &direntries,
3038
 
                                 notdotentries, alphasort);
3039
 
      if(numentries >= 0){
3040
 
        for(int i = 0; i < numentries; i++){
3041
 
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
3042
 
          if(ret == -1){
3043
 
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
3044
 
                         " \"%s\", 0): %s\n", tempdir,
3045
 
                         direntries[i]->d_name, strerror(errno));
3046
 
          }
3047
 
          free(direntries[i]);
3048
 
        }
3049
 
        
3050
 
        /* need to clean even if 0 because man page doesn't specify */
3051
 
        free(direntries);
3052
 
        if(numentries == -1){
3053
 
          perror_plus("scandir");
3054
 
        }
3055
 
        ret = rmdir(tempdir);
3056
 
        if(ret == -1 and errno != ENOENT){
3057
 
          perror_plus("rmdir");
3058
 
        }
3059
 
      }
3060
 
      close(tempdir_fd);
3061
 
    }
 
3113
    clean_dir_at(-1, tempdir, 0);
3062
3114
  }
3063
3115
  
3064
3116
  if(quit_now){