/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-16 23:01:02 UTC
  • Revision ID: teddy@recompile.se-20150716230102-62u5uwaguiufc6ac
mandos-client: Don't use memset().

* plugins.d/mandos-client.c (start_mandos_communication): Use compound
                                                          literal
                                                          struct
                                                          assignment
                                                          instead of
                                                          memset().

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() */
 
50
                                   asprintf(), strcpy() */
51
51
#include <sys/ioctl.h>          /* ioctl */
52
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
53
53
                                   sockaddr_in6, PF_INET6,
57
57
#include <sys/socket.h>         /* socket(), struct sockaddr_in6,
58
58
                                   inet_pton(), connect(),
59
59
                                   getnameinfo() */
60
 
#include <fcntl.h>              /* open(), unlinkat(), AT_REMOVEDIR */
 
60
#include <fcntl.h>              /* open(), unlinkat() */
61
61
#include <dirent.h>             /* opendir(), struct dirent, readdir()
62
62
                                 */
63
63
#include <inttypes.h>           /* PRIu16, PRIdMAX, intmax_t,
64
64
                                   strtoimax() */
65
 
#include <errno.h>              /* perror(), errno, EINTR, EINVAL,
66
 
                                   EAI_SYSTEM, ENETUNREACH,
67
 
                                   EHOSTUNREACH, ECONNREFUSED, EPROTO,
68
 
                                   EIO, ENOENT, ENXIO, ENOMEM, EISDIR,
69
 
                                   ENOTEMPTY,
 
65
#include <errno.h>              /* perror(), errno,
70
66
                                   program_invocation_short_name */
71
67
#include <time.h>               /* nanosleep(), time(), sleep() */
72
68
#include <net/if.h>             /* ioctl, ifreq, SIOCGIFFLAGS, IFF_UP,
309
305
      return false;
310
306
    }
311
307
    
312
 
    ret = close(fd);
 
308
    ret = (int)TEMP_FAILURE_RETRY(close(fd));
313
309
    if(ret == -1){
314
310
      perror_plus("close");
315
311
    }
517
513
  fprintf_plus(stderr, "GnuTLS: %s", string);
518
514
}
519
515
 
520
 
__attribute__((nonnull(1, 2, 4), warn_unused_result))
 
516
__attribute__((nonnull, warn_unused_result))
521
517
static int init_gnutls_global(const char *pubkeyfilename,
522
518
                              const char *seckeyfilename,
523
519
                              const char *dhparamsfilename,
529
525
    fprintf_plus(stderr, "Initializing GnuTLS\n");
530
526
  }
531
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
  
532
535
  if(debug){
533
536
    /* "Use a log level over 10 to enable all debugging options."
534
537
     * - GnuTLS manual
542
545
  if(ret != GNUTLS_E_SUCCESS){
543
546
    fprintf_plus(stderr, "GnuTLS memory error: %s\n",
544
547
                 safer_gnutls_strerror(ret));
 
548
    gnutls_global_deinit();
545
549
    return -1;
546
550
  }
547
551
  
751
755
 globalfail:
752
756
  
753
757
  gnutls_certificate_free_credentials(mc->cred);
 
758
  gnutls_global_deinit();
754
759
  gnutls_dh_params_deinit(mc->dh_params);
755
760
  return -1;
756
761
}
926
931
      perror_plus("dup2(devnull, STDIN_FILENO)");
927
932
      _exit(EX_OSERR);
928
933
    }
929
 
    ret = close(devnull);
 
934
    ret = (int)TEMP_FAILURE_RETRY(close(devnull));
930
935
    if(ret == -1){
931
936
      perror_plus("close");
932
937
      _exit(EX_OSERR);
949
954
                                                   helper, O_RDONLY));
950
955
    if(helper_fd == -1){
951
956
      perror_plus("openat");
952
 
      close(helperdir_fd);
953
957
      _exit(EX_UNAVAILABLE);
954
958
    }
955
 
    close(helperdir_fd);
 
959
    TEMP_FAILURE_RETRY(close(helperdir_fd));
956
960
#ifdef __GNUC__
957
961
#pragma GCC diagnostic push
958
962
#pragma GCC diagnostic ignored "-Wcast-qual"
1218
1222
                    sizeof(struct sockaddr_in));
1219
1223
    }
1220
1224
    if(ret < 0){
1221
 
      if(((errno == ENETUNREACH) or (errno == EHOSTUNREACH))
 
1225
      if(errno == ENETUNREACH
1222
1226
         and if_index != AVAHI_IF_UNSPEC
1223
1227
         and connect_to == NULL
1224
1228
         and not route_added and
1462
1466
    free(decrypted_buffer);
1463
1467
    free(buffer);
1464
1468
    if(tcp_sd >= 0){
1465
 
      ret = close(tcp_sd);
 
1469
      ret = (int)TEMP_FAILURE_RETRY(close(tcp_sd));
1466
1470
    }
1467
1471
    if(ret == -1){
1468
1472
      if(e == 0){
1632
1636
    errno = ret_errno;
1633
1637
    return false;
1634
1638
  }
1635
 
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
1636
 
  ifr->ifr_name[IF_NAMESIZE-1] = '\0'; /* NUL terminate */
 
1639
  strcpy(ifr->ifr_name, ifname);
1637
1640
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1638
1641
  if(ret == -1){
1639
1642
    if(debug){
1909
1912
      return;
1910
1913
    }
1911
1914
  }
 
1915
#ifdef __GLIBC__
 
1916
#if __GLIBC_PREREQ(2, 15)
1912
1917
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1913
1918
                           runnable_hook, alphasort);
 
1919
#else  /* not __GLIBC_PREREQ(2, 15) */
 
1920
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1921
                         alphasort);
 
1922
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
1923
#else   /* not __GLIBC__ */
 
1924
  int numhooks = scandir(hookdir, &direntries, runnable_hook,
 
1925
                         alphasort);
 
1926
#endif  /* not __GLIBC__ */
1914
1927
  if(numhooks == -1){
1915
1928
    perror_plus("scandir");
1916
1929
    return;
1998
2011
        perror_plus("openat");
1999
2012
        _exit(EXIT_FAILURE);
2000
2013
      }
2001
 
      if(close(hookdir_fd) == -1){
 
2014
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
2002
2015
        perror_plus("close");
2003
2016
        _exit(EXIT_FAILURE);
2004
2017
      }
2007
2020
        perror_plus("dup2(devnull, STDIN_FILENO)");
2008
2021
        _exit(EX_OSERR);
2009
2022
      }
2010
 
      ret = close(devnull);
 
2023
      ret = (int)TEMP_FAILURE_RETRY(close(devnull));
2011
2024
      if(ret == -1){
2012
2025
        perror_plus("close");
2013
2026
        _exit(EX_OSERR);
2062
2075
    free(direntry);
2063
2076
  }
2064
2077
  free(direntries);
2065
 
  if(close(hookdir_fd) == -1){
 
2078
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
2066
2079
    perror_plus("close");
2067
2080
  } else {
2068
2081
    hookdir_fd = -1;
2108
2121
    }
2109
2122
    
2110
2123
    if(quit_now){
2111
 
      ret = close(sd);
 
2124
      ret = (int)TEMP_FAILURE_RETRY(close(sd));
2112
2125
      if(ret == -1){
2113
2126
        perror_plus("close");
2114
2127
      }
2164
2177
    }
2165
2178
    
2166
2179
    /* Close the socket */
2167
 
    ret = close(sd);
 
2180
    ret = (int)TEMP_FAILURE_RETRY(close(sd));
2168
2181
    if(ret == -1){
2169
2182
      perror_plus("close");
2170
2183
    }
2252
2265
    }
2253
2266
    
2254
2267
    /* Close the socket */
2255
 
    int ret = close(sd);
 
2268
    int ret = (int)TEMP_FAILURE_RETRY(close(sd));
2256
2269
    if(ret == -1){
2257
2270
      perror_plus("close");
2258
2271
    }
2274
2287
 
2275
2288
int main(int argc, char *argv[]){
2276
2289
  mandos_context mc = { .server = NULL, .dh_bits = 0,
2277
 
                        .priority = "SECURE256:!CTYPE-X.509"
2278
 
                        ":+CTYPE-OPENPGP:!RSA:+SIGN-DSA-SHA256",
2279
 
                        .current_server = NULL, .interfaces = NULL,
2280
 
                        .interfaces_size = 0 };
 
2290
                        .priority = "SECURE256:!CTYPE-X.509:"
 
2291
                        "+CTYPE-OPENPGP:!RSA", .current_server = NULL,
 
2292
                        .interfaces = NULL, .interfaces_size = 0 };
2281
2293
  AvahiSServiceBrowser *sb = NULL;
2282
2294
  error_t ret_errno;
2283
2295
  int ret;
2510
2522
              }
2511
2523
            }
2512
2524
          }
2513
 
          close(seckey_fd);
 
2525
          TEMP_FAILURE_RETRY(close(seckey_fd));
2514
2526
        }
2515
2527
      }
2516
2528
      
2531
2543
              }
2532
2544
            }
2533
2545
          }
2534
 
          close(pubkey_fd);
 
2546
          TEMP_FAILURE_RETRY(close(pubkey_fd));
2535
2547
        }
2536
2548
      }
2537
2549
      
2538
 
      if(dh_params_file != NULL
2539
 
         and strcmp(dh_params_file, PATHDIR "/dhparams.pem" ) == 0){
 
2550
      if(strcmp(dh_params_file, PATHDIR "/dhparams.pem" ) == 0){
2540
2551
        int dhparams_fd = open(dh_params_file, O_RDONLY);
2541
2552
        if(dhparams_fd == -1){
2542
2553
          perror_plus("open");
2553
2564
              }
2554
2565
            }
2555
2566
          }
2556
 
          close(dhparams_fd);
 
2567
          TEMP_FAILURE_RETRY(close(dhparams_fd));
2557
2568
        }
2558
2569
      }
2559
2570
      
2961
2972
  
2962
2973
  if(gnutls_initialized){
2963
2974
    gnutls_certificate_free_credentials(mc.cred);
 
2975
    gnutls_global_deinit();
2964
2976
    gnutls_dh_params_deinit(mc.dh_params);
2965
2977
  }
2966
2978
  
3028
3040
  free(interfaces_to_take_down);
3029
3041
  free(interfaces_hooks);
3030
3042
  
3031
 
  void clean_dir_at(int base, const char * const dirname,
3032
 
                    uintmax_t level){
3033
 
    struct dirent **direntries = NULL;
3034
 
    int dret;
3035
 
    int dir_fd = (int)TEMP_FAILURE_RETRY(openat(base, dirname,
3036
 
                                                O_RDONLY
3037
 
                                                | O_NOFOLLOW
3038
 
                                                | O_DIRECTORY
3039
 
                                                | O_PATH));
3040
 
    if(dir_fd == -1){
3041
 
      perror_plus("open");
3042
 
    }
3043
 
    int numentries = scandirat(dir_fd, ".", &direntries,
3044
 
                               notdotentries, alphasort);
3045
 
    if(numentries >= 0){
3046
 
      for(int i = 0; i < numentries; i++){
3047
 
        if(debug){
3048
 
          fprintf_plus(stderr, "Unlinking \"%s/%s\"\n",
3049
 
                       dirname, direntries[i]->d_name);
3050
 
        }
3051
 
        dret = unlinkat(dir_fd, direntries[i]->d_name, 0);
3052
 
        if(dret == -1){
3053
 
          if(errno == EISDIR){
3054
 
              dret = unlinkat(dir_fd, direntries[i]->d_name,
3055
 
                              AT_REMOVEDIR);
3056
 
          }         
3057
 
          if((dret == -1) and (errno == ENOTEMPTY)
3058
 
             and (strcmp(direntries[i]->d_name, "private-keys-v1.d")
3059
 
                  == 0) and (level == 0)){
3060
 
            /* Recurse only in this special case */
3061
 
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3062
 
            dret = 0;
3063
 
          }
3064
 
          if(dret == -1){
3065
 
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3066
 
                         direntries[i]->d_name, strerror(errno));
3067
 
          }
3068
 
        }
3069
 
        free(direntries[i]);
3070
 
      }
3071
 
      
3072
 
      /* need to clean even if 0 because man page doesn't specify */
3073
 
      free(direntries);
3074
 
      if(numentries == -1){
3075
 
        perror_plus("scandirat");
3076
 
      }
3077
 
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3078
 
      if(dret == -1 and errno != ENOENT){
3079
 
        perror_plus("rmdir");
3080
 
      }
3081
 
    } else {
3082
 
      perror_plus("scandirat");
3083
 
    }
3084
 
    close(dir_fd);
3085
 
  }
3086
 
  
3087
3043
  /* Removes the GPGME temp directory and all files inside */
3088
3044
  if(tempdir != NULL){
3089
 
    clean_dir_at(-1, tempdir, 0);
 
3045
    struct dirent **direntries = NULL;
 
3046
    int tempdir_fd = (int)TEMP_FAILURE_RETRY(open(tempdir, O_RDONLY
 
3047
                                                  | O_NOFOLLOW
 
3048
                                                  | O_DIRECTORY
 
3049
                                                  | O_PATH));
 
3050
    if(tempdir_fd == -1){
 
3051
      perror_plus("open");
 
3052
    } else {
 
3053
#ifdef __GLIBC__
 
3054
#if __GLIBC_PREREQ(2, 15)
 
3055
      int numentries = scandirat(tempdir_fd, ".", &direntries,
 
3056
                                 notdotentries, alphasort);
 
3057
#else  /* not __GLIBC_PREREQ(2, 15) */
 
3058
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
3059
                               alphasort);
 
3060
#endif  /* not __GLIBC_PREREQ(2, 15) */
 
3061
#else   /* not __GLIBC__ */
 
3062
      int numentries = scandir(tempdir, &direntries, notdotentries,
 
3063
                               alphasort);
 
3064
#endif  /* not __GLIBC__ */
 
3065
      if(numentries >= 0){
 
3066
        for(int i = 0; i < numentries; i++){
 
3067
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
 
3068
          if(ret == -1){
 
3069
            fprintf_plus(stderr, "unlinkat(open(\"%s\", O_RDONLY),"
 
3070
                         " \"%s\", 0): %s\n", tempdir,
 
3071
                         direntries[i]->d_name, strerror(errno));
 
3072
          }
 
3073
          free(direntries[i]);
 
3074
        }
 
3075
        
 
3076
        /* need to clean even if 0 because man page doesn't specify */
 
3077
        free(direntries);
 
3078
        if(numentries == -1){
 
3079
          perror_plus("scandir");
 
3080
        }
 
3081
        ret = rmdir(tempdir);
 
3082
        if(ret == -1 and errno != ENOENT){
 
3083
          perror_plus("rmdir");
 
3084
        }
 
3085
      }
 
3086
      TEMP_FAILURE_RETRY(close(tempdir_fd));
 
3087
    }
3090
3088
  }
3091
3089
  
3092
3090
  if(quit_now){