/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

  • Committer: Teddy Hogeborn
  • Date: 2015-08-10 07:34:37 UTC
  • mto: (237.7.594 trunk)
  • mto: This revision was merged to the branch mainline in revision 325.
  • Revision ID: teddy@recompile.se-20150810073437-3m8jgt13nqric6vf
Revert change to D-Bus API.

The D-Bus API signal CheckerCompleted is documented to provide a
wait(2) status value.  Since the server switched to using subprocess
to run checkers, it no longer has access to a wait(2) status value.  A
previous change to work around this made the D-Bus API incompatible.
Revert this change by constructing a fake wait(2) status value; this
keeps the D-Bus API stable.

* DBUS-API (CheckerCompleted): Revert incompatible change.
* mandos (ClientDBus.checker_callback): Construct fake wait(2) status.
* mandos-monitor (MandosClientWidget.checker_completed): Revert to
                                                         using
                                                         original API
                                                         with wait(2)
                                                         condition
                                                         value.

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-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
 
12
 * Copyright © 2008-2015 Teddy Hogeborn
 
13
 * Copyright © 2008-2015 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
23
21
 * WITHOUT ANY WARRANTY; without even the implied warranty of
24
22
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
25
23
 * General Public License for more details.
26
24
 * 
27
25
 * You should have received a copy of the GNU General Public License
28
 
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
 
26
 * along with this program.  If not, see
 
27
 * <http://www.gnu.org/licenses/>.
29
28
 * 
30
29
 * Contact the authors at <mandos@recompile.se>.
31
30
 */
48
47
                                   strtof(), abort() */
49
48
#include <stdbool.h>            /* bool, false, true */
50
49
#include <string.h>             /* strcmp(), strlen(), strerror(),
51
 
                                   asprintf(), strncpy(), strsignal()
52
 
                                */
 
50
                                   asprintf(), strcpy() */
53
51
#include <sys/ioctl.h>          /* ioctl */
54
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
55
53
                                   sockaddr_in6, PF_INET6,
59
57
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
60
58
                                   inet_pton(), connect(),
61
59
                                   getnameinfo() */
62
 
#include <fcntl.h>              /* open(), unlinkat(), AT_REMOVEDIR */
 
60
#include <fcntl.h>              /* open(), unlinkat() */
63
61
#include <dirent.h>             /* opendir(), struct dirent, readdir()
64
62
                                 */
65
63
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
66
64
                                   strtoimax() */
67
 
#include <errno.h>              /* perror(), errno, EINTR, EINVAL,
68
 
                                   EAI_SYSTEM, ENETUNREACH,
69
 
                                   EHOSTUNREACH, ECONNREFUSED, EPROTO,
70
 
                                   EIO, ENOENT, ENXIO, ENOMEM, EISDIR,
71
 
                                   ENOTEMPTY,
 
65
#include <errno.h>              /* perror(), errno,
72
66
                                   program_invocation_short_name */
73
67
#include <time.h>               /* nanosleep(), time(), sleep() */
74
68
#include <net/if.h>             /* ioctl, ifreq, SIOCGIFFLAGS, IFF_UP,
519
513
  fprintf_plus(stderr, "GnuTLS: %s", string);
520
514
}
521
515
 
522
 
__attribute__((nonnull(1, 2, 4), warn_unused_result))
 
516
__attribute__((nonnull, warn_unused_result))
523
517
static int init_gnutls_global(const char *pubkeyfilename,
524
518
                              const char *seckeyfilename,
525
519
                              const char *dhparamsfilename,
531
525
    fprintf_plus(stderr, "Initializing GnuTLS\n");
532
526
  }
533
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
  
534
535
  if(debug){
535
536
    /* "Use a log level over 10 to enable all debugging options."
536
537
     * - GnuTLS manual
544
545
  if(ret != GNUTLS_E_SUCCESS){
545
546
    fprintf_plus(stderr, "GnuTLS memory error: %s\n",
546
547
                 safer_gnutls_strerror(ret));
 
548
    gnutls_global_deinit();
547
549
    return -1;
548
550
  }
549
551
  
613
615
        }
614
616
        params.size += (unsigned int)bytes_read;
615
617
      }
616
 
      ret = close(dhpfile);
617
 
      if(ret == -1){
618
 
        perror_plus("close");
619
 
      }
620
618
      if(params.data == NULL){
621
619
        dhparamsfilename = NULL;
622
620
      }
631
629
                     safer_gnutls_strerror(ret));
632
630
        dhparamsfilename = NULL;
633
631
      }
634
 
      free(params.data);
635
632
    } while(false);
636
633
  }
637
634
  if(dhparamsfilename == NULL){
758
755
 globalfail:
759
756
  
760
757
  gnutls_certificate_free_credentials(mc->cred);
 
758
  gnutls_global_deinit();
761
759
  gnutls_dh_params_deinit(mc->dh_params);
762
760
  return -1;
763
761
}
824
822
 
825
823
/* Set effective uid to 0, return errno */
826
824
__attribute__((warn_unused_result))
827
 
int raise_privileges(void){
828
 
  int old_errno = errno;
829
 
  int ret = 0;
 
825
error_t raise_privileges(void){
 
826
  error_t old_errno = errno;
 
827
  error_t ret_errno = 0;
830
828
  if(seteuid(0) == -1){
831
 
    ret = errno;
 
829
    ret_errno = errno;
832
830
  }
833
831
  errno = old_errno;
834
 
  return ret;
 
832
  return ret_errno;
835
833
}
836
834
 
837
835
/* Set effective and real user ID to 0.  Return errno. */
838
836
__attribute__((warn_unused_result))
839
 
int raise_privileges_permanently(void){
840
 
  int old_errno = errno;
841
 
  int ret = raise_privileges();
842
 
  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){
843
841
    errno = old_errno;
844
 
    return ret;
 
842
    return ret_errno;
845
843
  }
846
844
  if(setuid(0) == -1){
847
 
    ret = errno;
 
845
    ret_errno = errno;
848
846
  }
849
847
  errno = old_errno;
850
 
  return ret;
 
848
  return ret_errno;
851
849
}
852
850
 
853
851
/* Set effective user ID to unprivileged saved user ID */
854
852
__attribute__((warn_unused_result))
855
 
int lower_privileges(void){
856
 
  int old_errno = errno;
857
 
  int ret = 0;
 
853
error_t lower_privileges(void){
 
854
  error_t old_errno = errno;
 
855
  error_t ret_errno = 0;
858
856
  if(seteuid(uid) == -1){
859
 
    ret = errno;
 
857
    ret_errno = errno;
860
858
  }
861
859
  errno = old_errno;
862
 
  return ret;
 
860
  return ret_errno;
863
861
}
864
862
 
865
863
/* Lower privileges permanently */
866
864
__attribute__((warn_unused_result))
867
 
int lower_privileges_permanently(void){
868
 
  int old_errno = errno;
869
 
  int ret = 0;
 
865
error_t lower_privileges_permanently(void){
 
866
  error_t old_errno = errno;
 
867
  error_t ret_errno = 0;
870
868
  if(setuid(uid) == -1){
871
 
    ret = errno;
 
869
    ret_errno = errno;
872
870
  }
873
871
  errno = old_errno;
874
 
  return ret;
 
872
  return ret_errno;
875
873
}
876
874
 
877
875
/* Helper function to add_local_route() and delete_local_route() */
1084
1082
    bool match = false;
1085
1083
    {
1086
1084
      char *interface = NULL;
1087
 
      while((interface = argz_next(mc->interfaces,
1088
 
                                   mc->interfaces_size,
1089
 
                                   interface))){
 
1085
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
 
1086
                                 interface))){
1090
1087
        if(if_nametoindex(interface) == (unsigned int)if_index){
1091
1088
          match = true;
1092
1089
          break;
1226
1223
                    sizeof(struct sockaddr_in));
1227
1224
    }
1228
1225
    if(ret < 0){
1229
 
      if(((errno == ENETUNREACH) or (errno == EHOSTUNREACH))
 
1226
      if(errno == ENETUNREACH
1230
1227
         and if_index != AVAHI_IF_UNSPEC
1231
1228
         and connect_to == NULL
1232
1229
         and not route_added and
1245
1242
           with an explicit route added with the server's address.
1246
1243
           
1247
1244
           Avahi bug reference:
1248
 
           https://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1245
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
1249
1246
           https://bugs.debian.org/587961
1250
1247
        */
1251
1248
        if(debug){
1431
1428
                                               &decrypted_buffer, mc);
1432
1429
    if(decrypted_buffer_size >= 0){
1433
1430
      
1434
 
      clearerr(stdout);
1435
1431
      written = 0;
1436
1432
      while(written < (size_t) decrypted_buffer_size){
1437
1433
        if(quit_now){
1453
1449
        }
1454
1450
        written += (size_t)ret;
1455
1451
      }
1456
 
      ret = fflush(stdout);
1457
 
      if(ret != 0){
1458
 
        int e = errno;
1459
 
        if(debug){
1460
 
          fprintf_plus(stderr, "Error writing encrypted data: %s\n",
1461
 
                       strerror(errno));
1462
 
        }
1463
 
        errno = e;
1464
 
        goto mandos_end;
1465
 
      }
1466
1452
      retval = 0;
1467
1453
    }
1468
1454
  }
1499
1485
  return retval;
1500
1486
}
1501
1487
 
 
1488
__attribute__((nonnull))
1502
1489
static void resolve_callback(AvahiSServiceResolver *r,
1503
1490
                             AvahiIfIndex interface,
1504
1491
                             AvahiProtocol proto,
1641
1628
__attribute__((nonnull, warn_unused_result))
1642
1629
bool get_flags(const char *ifname, struct ifreq *ifr){
1643
1630
  int ret;
1644
 
  int old_errno;
 
1631
  error_t ret_errno;
1645
1632
  
1646
1633
  int s = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
1647
1634
  if(s < 0){
1648
 
    old_errno = errno;
 
1635
    ret_errno = errno;
1649
1636
    perror_plus("socket");
1650
 
    errno = old_errno;
 
1637
    errno = ret_errno;
1651
1638
    return false;
1652
1639
  }
1653
 
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1654
 
  ifr->ifr_name[IF_NAMESIZE-1] = '\0'; /* NUL terminate */
 
1640
  strcpy(ifr->ifr_name, ifname);
1655
1641
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1656
1642
  if(ret == -1){
1657
1643
    if(debug){
1658
 
      old_errno = errno;
 
1644
      ret_errno = errno;
1659
1645
      perror_plus("ioctl SIOCGIFFLAGS");
1660
 
      errno = old_errno;
1661
 
    }
1662
 
    if((close(s) == -1) and debug){
1663
 
      old_errno = errno;
1664
 
      perror_plus("close");
1665
 
      errno = old_errno;
 
1646
      errno = ret_errno;
1666
1647
    }
1667
1648
    return false;
1668
1649
  }
1669
 
  if((close(s) == -1) and debug){
1670
 
    old_errno = errno;
1671
 
    perror_plus("close");
1672
 
    errno = old_errno;
1673
 
  }
1674
1650
  return true;
1675
1651
}
1676
1652
 
1937
1913
      return;
1938
1914
    }
1939
1915
  }
1940
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1941
 
  if(devnull == -1){
1942
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1943
 
    return;
1944
 
  }
 
1916
#ifdef __GLIBC__
 
1917
#if __GLIBC_PREREQ(2, 15)
1945
1918
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1946
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__ */
1947
1928
  if(numhooks == -1){
1948
1929
    perror_plus("scandir");
1949
 
    close(devnull);
1950
1930
    return;
1951
1931
  }
1952
1932
  struct dirent *direntry;
1953
1933
  int ret;
 
1934
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1935
  if(devnull == -1){
 
1936
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1937
    return;
 
1938
  }
1954
1939
  for(int i = 0; i < numhooks; i++){
1955
1940
    direntry = direntries[i];
1956
1941
    if(debug){
2100
2085
}
2101
2086
 
2102
2087
__attribute__((nonnull, warn_unused_result))
2103
 
int bring_up_interface(const char *const interface,
2104
 
                       const float delay){
2105
 
  int old_errno = errno;
 
2088
error_t bring_up_interface(const char *const interface,
 
2089
                           const float delay){
 
2090
  error_t old_errno = errno;
2106
2091
  int ret;
2107
2092
  struct ifreq network;
2108
2093
  unsigned int if_index = if_nametoindex(interface);
2118
2103
  }
2119
2104
  
2120
2105
  if(not interface_is_up(interface)){
2121
 
    int ret_errno = 0;
2122
 
    int ioctl_errno = 0;
 
2106
    error_t ret_errno = 0, ioctl_errno = 0;
2123
2107
    if(not get_flags(interface, &network)){
2124
2108
      ret_errno = errno;
2125
2109
      fprintf_plus(stderr, "Failed to get flags for interface "
2212
2196
  
2213
2197
  /* Sleep checking until interface is running.
2214
2198
     Check every 0.25s, up to total time of delay */
2215
 
  for(int i = 0; i < delay * 4; i++){
 
2199
  for(int i=0; i < delay * 4; i++){
2216
2200
    if(interface_is_running(interface)){
2217
2201
      break;
2218
2202
    }
2228
2212
}
2229
2213
 
2230
2214
__attribute__((nonnull, warn_unused_result))
2231
 
int take_down_interface(const char *const interface){
2232
 
  int old_errno = errno;
 
2215
error_t take_down_interface(const char *const interface){
 
2216
  error_t old_errno = errno;
2233
2217
  struct ifreq network;
2234
2218
  unsigned int if_index = if_nametoindex(interface);
2235
2219
  if(if_index == 0){
2238
2222
    return ENXIO;
2239
2223
  }
2240
2224
  if(interface_is_up(interface)){
2241
 
    int ret_errno = 0;
2242
 
    int ioctl_errno = 0;
 
2225
    error_t ret_errno = 0, ioctl_errno = 0;
2243
2226
    if(not get_flags(interface, &network) and debug){
2244
2227
      ret_errno = errno;
2245
2228
      fprintf_plus(stderr, "Failed to get flags for interface "
2495
2478
                         .args_doc = "",
2496
2479
                         .doc = "Mandos client -- Get and decrypt"
2497
2480
                         " passwords from a Mandos server" };
2498
 
    ret_errno = argp_parse(&argp, argc, argv,
2499
 
                           ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
2500
 
    switch(ret_errno){
 
2481
    ret = argp_parse(&argp, argc, argv,
 
2482
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
2483
    switch(ret){
2501
2484
    case 0:
2502
2485
      break;
2503
2486
    case ENOMEM:
2504
2487
    default:
2505
 
      errno = ret_errno;
 
2488
      errno = ret;
2506
2489
      perror_plus("argp_parse");
2507
2490
      exitcode = EX_OSERR;
2508
2491
      goto end;
2514
2497
  
2515
2498
  {
2516
2499
    /* Work around Debian bug #633582:
2517
 
       <https://bugs.debian.org/633582> */
 
2500
       <http://bugs.debian.org/633582> */
2518
2501
    
2519
2502
    /* Re-raise privileges */
2520
 
    ret = raise_privileges();
2521
 
    if(ret != 0){
2522
 
      errno = ret;
 
2503
    ret_errno = raise_privileges();
 
2504
    if(ret_errno != 0){
 
2505
      errno = ret_errno;
2523
2506
      perror_plus("Failed to raise privileges");
2524
2507
    } else {
2525
2508
      struct stat st;
2589
2572
      }
2590
2573
      
2591
2574
      /* Lower privileges */
2592
 
      ret = lower_privileges();
2593
 
      if(ret != 0){
2594
 
        errno = ret;
 
2575
      ret_errno = lower_privileges();
 
2576
      if(ret_errno != 0){
 
2577
        errno = ret_errno;
2595
2578
        perror_plus("Failed to lower privileges");
2596
2579
      }
2597
2580
    }
2925
2908
    
2926
2909
    /* Allocate a new server */
2927
2910
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
2928
 
                                 &config, NULL, NULL, &ret);
 
2911
                                 &config, NULL, NULL, &ret_errno);
2929
2912
    
2930
2913
    /* Free the Avahi configuration data */
2931
2914
    avahi_server_config_free(&config);
2934
2917
  /* Check if creating the Avahi server object succeeded */
2935
2918
  if(mc.server == NULL){
2936
2919
    fprintf_plus(stderr, "Failed to create Avahi server: %s\n",
2937
 
                 avahi_strerror(ret));
 
2920
                 avahi_strerror(ret_errno));
2938
2921
    exitcode = EX_UNAVAILABLE;
2939
2922
    goto end;
2940
2923
  }
2975
2958
 end:
2976
2959
  
2977
2960
  if(debug){
2978
 
    if(signal_received){
2979
 
      fprintf_plus(stderr, "%s exiting due to signal %d: %s\n",
2980
 
                   argv[0], signal_received,
2981
 
                   strsignal(signal_received));
2982
 
    } else {
2983
 
      fprintf_plus(stderr, "%s exiting\n", argv[0]);
2984
 
    }
 
2961
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
2985
2962
  }
2986
2963
  
2987
2964
  /* Cleanup things */
2998
2975
  
2999
2976
  if(gnutls_initialized){
3000
2977
    gnutls_certificate_free_credentials(mc.cred);
 
2978
    gnutls_global_deinit();
3001
2979
    gnutls_dh_params_deinit(mc.dh_params);
3002
2980
  }
3003
2981
  
3026
3004
  
3027
3005
  /* Re-raise privileges */
3028
3006
  {
3029
 
    ret = raise_privileges();
3030
 
    if(ret != 0){
3031
 
      errno = ret;
 
3007
    ret_errno = raise_privileges();
 
3008
    if(ret_errno != 0){
 
3009
      errno = ret_errno;
3032
3010
      perror_plus("Failed to raise privileges");
3033
3011
    } else {
3034
3012
      
3039
3017
      /* Take down the network interfaces which were brought up */
3040
3018
      {
3041
3019
        char *interface = NULL;
3042
 
        while((interface = argz_next(interfaces_to_take_down,
3043
 
                                     interfaces_to_take_down_size,
3044
 
                                     interface))){
3045
 
          ret = take_down_interface(interface);
3046
 
          if(ret != 0){
3047
 
            errno = ret;
 
3020
        while((interface=argz_next(interfaces_to_take_down,
 
3021
                                   interfaces_to_take_down_size,
 
3022
                                   interface))){
 
3023
          ret_errno = take_down_interface(interface);
 
3024
          if(ret_errno != 0){
 
3025
            errno = ret_errno;
3048
3026
            perror_plus("Failed to take down interface");
3049
3027
          }
3050
3028
        }
3055
3033
      }
3056
3034
    }
3057
3035
    
3058
 
    ret = lower_privileges_permanently();
3059
 
    if(ret != 0){
3060
 
      errno = ret;
 
3036
    ret_errno = lower_privileges_permanently();
 
3037
    if(ret_errno != 0){
 
3038
      errno = ret_errno;
3061
3039
      perror_plus("Failed to lower privileges permanently");
3062
3040
    }
3063
3041
  }
3065
3043
  free(interfaces_to_take_down);
3066
3044
  free(interfaces_hooks);
3067
3045
  
3068
 
  void clean_dir_at(int base, const char * const dirname,
3069
 
                    uintmax_t level){
3070
 
    struct dirent **direntries = NULL;
3071
 
    int dret;
3072
 
    int dir_fd = (int)TEMP_FAILURE_RETRY(openat(base, dirname,
3073
 
                                                O_RDONLY
3074
 
                                                | O_NOFOLLOW
3075
 
                                                | O_DIRECTORY
3076
 
                                                | O_PATH));
3077
 
    if(dir_fd == -1){
3078
 
      perror_plus("open");
3079
 
      return;
3080
 
    }
3081
 
    int numentries = scandirat(dir_fd, ".", &direntries,
3082
 
                               notdotentries, alphasort);
3083
 
    if(numentries >= 0){
3084
 
      for(int i = 0; i < numentries; i++){
3085
 
        if(debug){
3086
 
          fprintf_plus(stderr, "Unlinking \"%s/%s\"\n",
3087
 
                       dirname, direntries[i]->d_name);
3088
 
        }
3089
 
        dret = unlinkat(dir_fd, direntries[i]->d_name, 0);
3090
 
        if(dret == -1){
3091
 
          if(errno == EISDIR){
3092
 
              dret = unlinkat(dir_fd, direntries[i]->d_name,
3093
 
                              AT_REMOVEDIR);
3094
 
          }         
3095
 
          if((dret == -1) and (errno == ENOTEMPTY)
3096
 
             and (strcmp(direntries[i]->d_name, "private-keys-v1.d")
3097
 
                  == 0) and (level == 0)){
3098
 
            /* Recurse only in this special case */
3099
 
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3100
 
            dret = 0;
3101
 
          }
3102
 
          if((dret == -1) and (errno != ENOENT)){
3103
 
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3104
 
                         direntries[i]->d_name, strerror(errno));
3105
 
          }
3106
 
        }
3107
 
        free(direntries[i]);
3108
 
      }
3109
 
      
3110
 
      /* need to clean even if 0 because man page doesn't specify */
3111
 
      free(direntries);
3112
 
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3113
 
      if(dret == -1 and errno != ENOENT){
3114
 
        perror_plus("rmdir");
3115
 
      }
3116
 
    } else {
3117
 
      perror_plus("scandirat");
3118
 
    }
3119
 
    close(dir_fd);
3120
 
  }
3121
 
  
3122
3046
  /* Removes the GPGME temp directory and all files inside */
3123
3047
  if(tempdir != NULL){
3124
 
    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
    }
3125
3091
  }
3126
3092
  
3127
3093
  if(quit_now){