/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-03-04 22:07:35 UTC
  • Revision ID: teddy@recompile.se-20160304220735-4xeeqt5p4nhw5cuh
Restrict the Mandos server daemon in the systemd service file.

* mandos.service ([Service]/ProtectSystem): Set to "full".
 ([Service]/PrivateTmp, [Service]/PrivateDevices,
  [Service]/ProtectHome): Set to "yes".
 ([Service]/CapabilityBoundingSet): Set to "CAP_SETUID
                                    CAP_DAC_OVERRIDE CAP_NET_RAW".

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-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
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(), strncpy() */
53
51
#include <sys/ioctl.h>          /* ioctl */
54
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
55
53
                                   sockaddr_in6, PF_INET6,
310
308
                   gpgme_strsource(rc), gpgme_strerror(rc));
311
309
      return false;
312
310
    }
313
 
    {
314
 
      gpgme_import_result_t import_result
315
 
        = gpgme_op_import_result(mc->ctx);
316
 
      if((import_result->imported < 1
317
 
          or import_result->not_imported > 0)
318
 
         and import_result->unchanged == 0){
319
 
        fprintf_plus(stderr, "bad gpgme_op_import_results:\n");
320
 
        fprintf_plus(stderr,
321
 
                     "The total number of considered keys: %d\n",
322
 
                     import_result->considered);
323
 
        fprintf_plus(stderr,
324
 
                     "The number of keys without user ID: %d\n",
325
 
                     import_result->no_user_id);
326
 
        fprintf_plus(stderr,
327
 
                     "The total number of imported keys: %d\n",
328
 
                     import_result->imported);
329
 
        fprintf_plus(stderr, "The number of imported RSA keys: %d\n",
330
 
                     import_result->imported_rsa);
331
 
        fprintf_plus(stderr, "The number of unchanged keys: %d\n",
332
 
                     import_result->unchanged);
333
 
        fprintf_plus(stderr, "The number of new user IDs: %d\n",
334
 
                     import_result->new_user_ids);
335
 
        fprintf_plus(stderr, "The number of new sub keys: %d\n",
336
 
                     import_result->new_sub_keys);
337
 
        fprintf_plus(stderr, "The number of new signatures: %d\n",
338
 
                     import_result->new_signatures);
339
 
        fprintf_plus(stderr, "The number of new revocations: %d\n",
340
 
                     import_result->new_revocations);
341
 
        fprintf_plus(stderr,
342
 
                     "The total number of secret keys read: %d\n",
343
 
                     import_result->secret_read);
344
 
        fprintf_plus(stderr,
345
 
                     "The number of imported secret keys: %d\n",
346
 
                     import_result->secret_imported);
347
 
        fprintf_plus(stderr,
348
 
                     "The number of unchanged secret keys: %d\n",
349
 
                     import_result->secret_unchanged);
350
 
        fprintf_plus(stderr, "The number of keys not imported: %d\n",
351
 
                     import_result->not_imported);
352
 
        for(gpgme_import_status_t import_status
353
 
              = import_result->imports;
354
 
            import_status != NULL;
355
 
            import_status = import_status->next){
356
 
          fprintf_plus(stderr, "Import status for key: %s\n",
357
 
                       import_status->fpr);
358
 
          if(import_status->result != GPG_ERR_NO_ERROR){
359
 
            fprintf_plus(stderr, "Import result: %s: %s\n",
360
 
                         gpgme_strsource(import_status->result),
361
 
                         gpgme_strerror(import_status->result));
362
 
          }
363
 
          fprintf_plus(stderr, "Key status:\n");
364
 
          fprintf_plus(stderr,
365
 
                       import_status->status & GPGME_IMPORT_NEW
366
 
                       ? "The key was new.\n"
367
 
                       : "The key was not new.\n");
368
 
          fprintf_plus(stderr,
369
 
                       import_status->status & GPGME_IMPORT_UID
370
 
                       ? "The key contained new user IDs.\n"
371
 
                       : "The key did not contain new user IDs.\n");
372
 
          fprintf_plus(stderr,
373
 
                       import_status->status & GPGME_IMPORT_SIG
374
 
                       ? "The key contained new signatures.\n"
375
 
                       : "The key did not contain new signatures.\n");
376
 
          fprintf_plus(stderr,
377
 
                       import_status->status & GPGME_IMPORT_SUBKEY
378
 
                       ? "The key contained new sub keys.\n"
379
 
                       : "The key did not contain new sub keys.\n");
380
 
          fprintf_plus(stderr,
381
 
                       import_status->status & GPGME_IMPORT_SECRET
382
 
                       ? "The key contained a secret key.\n"
383
 
                       : "The key did not contain a secret key.\n");
384
 
        }
385
 
        return false;
386
 
      }
387
 
    }
388
311
    
389
312
    ret = close(fd);
390
313
    if(ret == -1){
431
354
  /* Create new GPGME "context" */
432
355
  rc = gpgme_new(&(mc->ctx));
433
356
  if(rc != GPG_ERR_NO_ERROR){
434
 
    fprintf_plus(stderr, "bad gpgme_new: %s: %s\n",
435
 
                 gpgme_strsource(rc), gpgme_strerror(rc));
 
357
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
358
                 "bad gpgme_new: %s: %s\n", gpgme_strsource(rc),
 
359
                 gpgme_strerror(rc));
436
360
    return false;
437
361
  }
438
362
  
474
398
  /* Create new empty GPGME data buffer for the plaintext */
475
399
  rc = gpgme_data_new(&dh_plain);
476
400
  if(rc != GPG_ERR_NO_ERROR){
477
 
    fprintf_plus(stderr, "bad gpgme_data_new: %s: %s\n",
 
401
    fprintf_plus(stderr, "Mandos plugin mandos-client: "
 
402
                 "bad gpgme_data_new: %s: %s\n",
478
403
                 gpgme_strsource(rc), gpgme_strerror(rc));
479
404
    gpgme_data_release(dh_crypto);
480
405
    return -1;
493
418
      if(result == NULL){
494
419
        fprintf_plus(stderr, "gpgme_op_decrypt_result failed\n");
495
420
      } else {
496
 
        if(result->unsupported_algorithm != NULL) {
497
 
          fprintf_plus(stderr, "Unsupported algorithm: %s\n",
498
 
                       result->unsupported_algorithm);
499
 
        }
500
 
        fprintf_plus(stderr, "Wrong key usage: %s\n",
501
 
                     result->wrong_key_usage ? "Yes" : "No");
 
421
        fprintf_plus(stderr, "Unsupported algorithm: %s\n",
 
422
                     result->unsupported_algorithm);
 
423
        fprintf_plus(stderr, "Wrong key usage: %u\n",
 
424
                     result->wrong_key_usage);
502
425
        if(result->file_name != NULL){
503
426
          fprintf_plus(stderr, "File name: %s\n", result->file_name);
504
427
        }
505
 
 
506
 
        for(gpgme_recipient_t r = result->recipients; r != NULL;
507
 
            r = r->next){
 
428
        gpgme_recipient_t recipient;
 
429
        recipient = result->recipients;
 
430
        while(recipient != NULL){
508
431
          fprintf_plus(stderr, "Public key algorithm: %s\n",
509
 
                       gpgme_pubkey_algo_name(r->pubkey_algo));
510
 
          fprintf_plus(stderr, "Key ID: %s\n", r->keyid);
 
432
                       gpgme_pubkey_algo_name
 
433
                       (recipient->pubkey_algo));
 
434
          fprintf_plus(stderr, "Key ID: %s\n", recipient->keyid);
511
435
          fprintf_plus(stderr, "Secret key available: %s\n",
512
 
                       r->status == GPG_ERR_NO_SECKEY ? "No" : "Yes");
 
436
                       recipient->status == GPG_ERR_NO_SECKEY
 
437
                       ? "No" : "Yes");
 
438
          recipient = recipient->next;
513
439
        }
514
440
      }
515
441
    }
685
611
        }
686
612
        params.size += (unsigned int)bytes_read;
687
613
      }
688
 
      ret = close(dhpfile);
689
 
      if(ret == -1){
690
 
        perror_plus("close");
691
 
      }
692
614
      if(params.data == NULL){
693
615
        dhparamsfilename = NULL;
694
616
      }
703
625
                     safer_gnutls_strerror(ret));
704
626
        dhparamsfilename = NULL;
705
627
      }
706
 
      free(params.data);
707
628
    } while(false);
708
629
  }
709
630
  if(dhparamsfilename == NULL){
896
817
 
897
818
/* Set effective uid to 0, return errno */
898
819
__attribute__((warn_unused_result))
899
 
int raise_privileges(void){
900
 
  int old_errno = errno;
901
 
  int ret = 0;
 
820
error_t raise_privileges(void){
 
821
  error_t old_errno = errno;
 
822
  error_t ret_errno = 0;
902
823
  if(seteuid(0) == -1){
903
 
    ret = errno;
 
824
    ret_errno = errno;
904
825
  }
905
826
  errno = old_errno;
906
 
  return ret;
 
827
  return ret_errno;
907
828
}
908
829
 
909
830
/* Set effective and real user ID to 0.  Return errno. */
910
831
__attribute__((warn_unused_result))
911
 
int raise_privileges_permanently(void){
912
 
  int old_errno = errno;
913
 
  int ret = raise_privileges();
914
 
  if(ret != 0){
 
832
error_t raise_privileges_permanently(void){
 
833
  error_t old_errno = errno;
 
834
  error_t ret_errno = raise_privileges();
 
835
  if(ret_errno != 0){
915
836
    errno = old_errno;
916
 
    return ret;
 
837
    return ret_errno;
917
838
  }
918
839
  if(setuid(0) == -1){
919
 
    ret = errno;
 
840
    ret_errno = errno;
920
841
  }
921
842
  errno = old_errno;
922
 
  return ret;
 
843
  return ret_errno;
923
844
}
924
845
 
925
846
/* Set effective user ID to unprivileged saved user ID */
926
847
__attribute__((warn_unused_result))
927
 
int lower_privileges(void){
928
 
  int old_errno = errno;
929
 
  int ret = 0;
 
848
error_t lower_privileges(void){
 
849
  error_t old_errno = errno;
 
850
  error_t ret_errno = 0;
930
851
  if(seteuid(uid) == -1){
931
 
    ret = errno;
 
852
    ret_errno = errno;
932
853
  }
933
854
  errno = old_errno;
934
 
  return ret;
 
855
  return ret_errno;
935
856
}
936
857
 
937
858
/* Lower privileges permanently */
938
859
__attribute__((warn_unused_result))
939
 
int lower_privileges_permanently(void){
940
 
  int old_errno = errno;
941
 
  int ret = 0;
 
860
error_t lower_privileges_permanently(void){
 
861
  error_t old_errno = errno;
 
862
  error_t ret_errno = 0;
942
863
  if(setuid(uid) == -1){
943
 
    ret = errno;
 
864
    ret_errno = errno;
944
865
  }
945
866
  errno = old_errno;
946
 
  return ret;
 
867
  return ret_errno;
947
868
}
948
869
 
949
870
/* Helper function to add_local_route() and delete_local_route() */
1156
1077
    bool match = false;
1157
1078
    {
1158
1079
      char *interface = NULL;
1159
 
      while((interface = argz_next(mc->interfaces,
1160
 
                                   mc->interfaces_size,
1161
 
                                   interface))){
 
1080
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
 
1081
                                 interface))){
1162
1082
        if(if_nametoindex(interface) == (unsigned int)if_index){
1163
1083
          match = true;
1164
1084
          break;
1317
1237
           with an explicit route added with the server's address.
1318
1238
           
1319
1239
           Avahi bug reference:
1320
 
           https://lists.freedesktop.org/archives/avahi/2010-February/001833.html
 
1240
           http://lists.freedesktop.org/archives/avahi/2010-February/001833.html
1321
1241
           https://bugs.debian.org/587961
1322
1242
        */
1323
1243
        if(debug){
1503
1423
                                               &decrypted_buffer, mc);
1504
1424
    if(decrypted_buffer_size >= 0){
1505
1425
      
1506
 
      clearerr(stdout);
1507
1426
      written = 0;
1508
1427
      while(written < (size_t) decrypted_buffer_size){
1509
1428
        if(quit_now){
1525
1444
        }
1526
1445
        written += (size_t)ret;
1527
1446
      }
1528
 
      ret = fflush(stdout);
1529
 
      if(ret != 0){
1530
 
        int e = errno;
1531
 
        if(debug){
1532
 
          fprintf_plus(stderr, "Error writing encrypted data: %s\n",
1533
 
                       strerror(errno));
1534
 
        }
1535
 
        errno = e;
1536
 
        goto mandos_end;
1537
 
      }
1538
1447
      retval = 0;
1539
1448
    }
1540
1449
  }
1571
1480
  return retval;
1572
1481
}
1573
1482
 
 
1483
__attribute__((nonnull))
1574
1484
static void resolve_callback(AvahiSServiceResolver *r,
1575
1485
                             AvahiIfIndex interface,
1576
1486
                             AvahiProtocol proto,
1713
1623
__attribute__((nonnull, warn_unused_result))
1714
1624
bool get_flags(const char *ifname, struct ifreq *ifr){
1715
1625
  int ret;
1716
 
  int old_errno;
 
1626
  error_t ret_errno;
1717
1627
  
1718
1628
  int s = socket(PF_INET6, SOCK_DGRAM, IPPROTO_IP);
1719
1629
  if(s < 0){
1720
 
    old_errno = errno;
 
1630
    ret_errno = errno;
1721
1631
    perror_plus("socket");
1722
 
    errno = old_errno;
 
1632
    errno = ret_errno;
1723
1633
    return false;
1724
1634
  }
1725
1635
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1727
1637
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1728
1638
  if(ret == -1){
1729
1639
    if(debug){
1730
 
      old_errno = errno;
 
1640
      ret_errno = errno;
1731
1641
      perror_plus("ioctl SIOCGIFFLAGS");
1732
 
      errno = old_errno;
1733
 
    }
1734
 
    if((close(s) == -1) and debug){
1735
 
      old_errno = errno;
1736
 
      perror_plus("close");
1737
 
      errno = old_errno;
 
1642
      errno = ret_errno;
1738
1643
    }
1739
1644
    return false;
1740
1645
  }
1741
 
  if((close(s) == -1) and debug){
1742
 
    old_errno = errno;
1743
 
    perror_plus("close");
1744
 
    errno = old_errno;
1745
 
  }
1746
1646
  return true;
1747
1647
}
1748
1648
 
2009
1909
      return;
2010
1910
    }
2011
1911
  }
2012
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
2013
 
  if(devnull == -1){
2014
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
2015
 
    return;
2016
 
  }
2017
1912
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
2018
1913
                           runnable_hook, alphasort);
2019
1914
  if(numhooks == -1){
2020
1915
    perror_plus("scandir");
2021
 
    close(devnull);
2022
1916
    return;
2023
1917
  }
2024
1918
  struct dirent *direntry;
2025
1919
  int ret;
 
1920
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1921
  if(devnull == -1){
 
1922
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1923
    return;
 
1924
  }
2026
1925
  for(int i = 0; i < numhooks; i++){
2027
1926
    direntry = direntries[i];
2028
1927
    if(debug){
2172
2071
}
2173
2072
 
2174
2073
__attribute__((nonnull, warn_unused_result))
2175
 
int bring_up_interface(const char *const interface,
2176
 
                       const float delay){
2177
 
  int old_errno = errno;
 
2074
error_t bring_up_interface(const char *const interface,
 
2075
                           const float delay){
 
2076
  error_t old_errno = errno;
2178
2077
  int ret;
2179
2078
  struct ifreq network;
2180
2079
  unsigned int if_index = if_nametoindex(interface);
2190
2089
  }
2191
2090
  
2192
2091
  if(not interface_is_up(interface)){
2193
 
    int ret_errno = 0;
2194
 
    int ioctl_errno = 0;
 
2092
    error_t ret_errno = 0, ioctl_errno = 0;
2195
2093
    if(not get_flags(interface, &network)){
2196
2094
      ret_errno = errno;
2197
2095
      fprintf_plus(stderr, "Failed to get flags for interface "
2284
2182
  
2285
2183
  /* Sleep checking until interface is running.
2286
2184
     Check every 0.25s, up to total time of delay */
2287
 
  for(int i = 0; i < delay * 4; i++){
 
2185
  for(int i=0; i < delay * 4; i++){
2288
2186
    if(interface_is_running(interface)){
2289
2187
      break;
2290
2188
    }
2300
2198
}
2301
2199
 
2302
2200
__attribute__((nonnull, warn_unused_result))
2303
 
int take_down_interface(const char *const interface){
2304
 
  int old_errno = errno;
 
2201
error_t take_down_interface(const char *const interface){
 
2202
  error_t old_errno = errno;
2305
2203
  struct ifreq network;
2306
2204
  unsigned int if_index = if_nametoindex(interface);
2307
2205
  if(if_index == 0){
2310
2208
    return ENXIO;
2311
2209
  }
2312
2210
  if(interface_is_up(interface)){
2313
 
    int ret_errno = 0;
2314
 
    int ioctl_errno = 0;
 
2211
    error_t ret_errno = 0, ioctl_errno = 0;
2315
2212
    if(not get_flags(interface, &network) and debug){
2316
2213
      ret_errno = errno;
2317
2214
      fprintf_plus(stderr, "Failed to get flags for interface "
2567
2464
                         .args_doc = "",
2568
2465
                         .doc = "Mandos client -- Get and decrypt"
2569
2466
                         " passwords from a Mandos server" };
2570
 
    ret_errno = argp_parse(&argp, argc, argv,
2571
 
                           ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
2572
 
    switch(ret_errno){
 
2467
    ret = argp_parse(&argp, argc, argv,
 
2468
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
2469
    switch(ret){
2573
2470
    case 0:
2574
2471
      break;
2575
2472
    case ENOMEM:
2576
2473
    default:
2577
 
      errno = ret_errno;
 
2474
      errno = ret;
2578
2475
      perror_plus("argp_parse");
2579
2476
      exitcode = EX_OSERR;
2580
2477
      goto end;
2586
2483
  
2587
2484
  {
2588
2485
    /* Work around Debian bug #633582:
2589
 
       <https://bugs.debian.org/633582> */
 
2486
       <http://bugs.debian.org/633582> */
2590
2487
    
2591
2488
    /* Re-raise privileges */
2592
 
    ret = raise_privileges();
2593
 
    if(ret != 0){
2594
 
      errno = ret;
 
2489
    ret_errno = raise_privileges();
 
2490
    if(ret_errno != 0){
 
2491
      errno = ret_errno;
2595
2492
      perror_plus("Failed to raise privileges");
2596
2493
    } else {
2597
2494
      struct stat st;
2661
2558
      }
2662
2559
      
2663
2560
      /* Lower privileges */
2664
 
      ret = lower_privileges();
2665
 
      if(ret != 0){
2666
 
        errno = ret;
 
2561
      ret_errno = lower_privileges();
 
2562
      if(ret_errno != 0){
 
2563
        errno = ret_errno;
2667
2564
        perror_plus("Failed to lower privileges");
2668
2565
      }
2669
2566
    }
2997
2894
    
2998
2895
    /* Allocate a new server */
2999
2896
    mc.server = avahi_server_new(avahi_simple_poll_get(simple_poll),
3000
 
                                 &config, NULL, NULL, &ret);
 
2897
                                 &config, NULL, NULL, &ret_errno);
3001
2898
    
3002
2899
    /* Free the Avahi configuration data */
3003
2900
    avahi_server_config_free(&config);
3006
2903
  /* Check if creating the Avahi server object succeeded */
3007
2904
  if(mc.server == NULL){
3008
2905
    fprintf_plus(stderr, "Failed to create Avahi server: %s\n",
3009
 
                 avahi_strerror(ret));
 
2906
                 avahi_strerror(ret_errno));
3010
2907
    exitcode = EX_UNAVAILABLE;
3011
2908
    goto end;
3012
2909
  }
3047
2944
 end:
3048
2945
  
3049
2946
  if(debug){
3050
 
    if(signal_received){
3051
 
      fprintf_plus(stderr, "%s exiting due to signal %d: %s\n",
3052
 
                   argv[0], signal_received,
3053
 
                   strsignal(signal_received));
3054
 
    } else {
3055
 
      fprintf_plus(stderr, "%s exiting\n", argv[0]);
3056
 
    }
 
2947
    fprintf_plus(stderr, "%s exiting\n", argv[0]);
3057
2948
  }
3058
2949
  
3059
2950
  /* Cleanup things */
3098
2989
  
3099
2990
  /* Re-raise privileges */
3100
2991
  {
3101
 
    ret = raise_privileges();
3102
 
    if(ret != 0){
3103
 
      errno = ret;
 
2992
    ret_errno = raise_privileges();
 
2993
    if(ret_errno != 0){
 
2994
      errno = ret_errno;
3104
2995
      perror_plus("Failed to raise privileges");
3105
2996
    } else {
3106
2997
      
3111
3002
      /* Take down the network interfaces which were brought up */
3112
3003
      {
3113
3004
        char *interface = NULL;
3114
 
        while((interface = argz_next(interfaces_to_take_down,
3115
 
                                     interfaces_to_take_down_size,
3116
 
                                     interface))){
3117
 
          ret = take_down_interface(interface);
3118
 
          if(ret != 0){
3119
 
            errno = ret;
 
3005
        while((interface=argz_next(interfaces_to_take_down,
 
3006
                                   interfaces_to_take_down_size,
 
3007
                                   interface))){
 
3008
          ret_errno = take_down_interface(interface);
 
3009
          if(ret_errno != 0){
 
3010
            errno = ret_errno;
3120
3011
            perror_plus("Failed to take down interface");
3121
3012
          }
3122
3013
        }
3127
3018
      }
3128
3019
    }
3129
3020
    
3130
 
    ret = lower_privileges_permanently();
3131
 
    if(ret != 0){
3132
 
      errno = ret;
 
3021
    ret_errno = lower_privileges_permanently();
 
3022
    if(ret_errno != 0){
 
3023
      errno = ret_errno;
3133
3024
      perror_plus("Failed to lower privileges permanently");
3134
3025
    }
3135
3026
  }
3148
3039
                                                | O_PATH));
3149
3040
    if(dir_fd == -1){
3150
3041
      perror_plus("open");
3151
 
      return;
3152
3042
    }
3153
3043
    int numentries = scandirat(dir_fd, ".", &direntries,
3154
3044
                               notdotentries, alphasort);
3171
3061
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3172
3062
            dret = 0;
3173
3063
          }
3174
 
          if((dret == -1) and (errno != ENOENT)){
 
3064
          if(dret == -1){
3175
3065
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3176
3066
                         direntries[i]->d_name, strerror(errno));
3177
3067
          }
3181
3071
      
3182
3072
      /* need to clean even if 0 because man page doesn't specify */
3183
3073
      free(direntries);
 
3074
      if(numentries == -1){
 
3075
        perror_plus("scandirat");
 
3076
      }
3184
3077
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3185
3078
      if(dret == -1 and errno != ENOENT){
3186
3079
        perror_plus("rmdir");