/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: 2016-02-21 12:55:39 UTC
  • mto: (237.7.594 trunk)
  • mto: This revision was merged to the branch mainline in revision 331.
  • Revision ID: teddy@recompile.se-20160221125539-wi1n8ci0ldbdxrar
plugins.d/mandos-client.c (get_flags): Use strncpy(), not strcpy().

* plugins.d/mandos-client.c (get_flags): Use strncpy() instead of
  strcpy().  Note that as the code works currently, this is never a
  problem in practice, since only interface names which were valid
  existing interface names (and therefore had names with lengths below
  IF_NAMESIZE) is ever passed to the get_flags() function.  But it
  doesn't hurt to check, anyway.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
46
46
#include <stdlib.h>             /* free(), EXIT_SUCCESS, srand(),
47
47
                                   strtof(), abort() */
48
48
#include <stdbool.h>            /* bool, false, true */
49
 
#include <string.h>             /* memset(), strcmp(), strlen(),
50
 
                                   strerror(), asprintf(), strcpy() */
 
49
#include <string.h>             /* strcmp(), strlen(), strerror(),
 
50
                                   asprintf(), strncpy() */
51
51
#include <sys/ioctl.h>          /* ioctl */
52
52
#include <sys/types.h>          /* socket(), inet_pton(), sockaddr,
53
53
                                   sockaddr_in6, PF_INET6,
305
305
      return false;
306
306
    }
307
307
    
308
 
    ret = (int)TEMP_FAILURE_RETRY(close(fd));
 
308
    ret = close(fd);
309
309
    if(ret == -1){
310
310
      perror_plus("close");
311
311
    }
931
931
      perror_plus("dup2(devnull, STDIN_FILENO)");
932
932
      _exit(EX_OSERR);
933
933
    }
934
 
    ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
934
    ret = close(devnull);
935
935
    if(ret == -1){
936
936
      perror_plus("close");
937
937
      _exit(EX_OSERR);
954
954
                                                   helper, O_RDONLY));
955
955
    if(helper_fd == -1){
956
956
      perror_plus("openat");
 
957
      close(helperdir_fd);
957
958
      _exit(EX_UNAVAILABLE);
958
959
    }
959
 
    TEMP_FAILURE_RETRY(close(helperdir_fd));
 
960
    close(helperdir_fd);
960
961
#ifdef __GNUC__
961
962
#pragma GCC diagnostic push
962
963
#pragma GCC diagnostic ignored "-Wcast-qual"
1130
1131
    goto mandos_end;
1131
1132
  }
1132
1133
  
1133
 
  memset(&to, 0, sizeof(to));
1134
1134
  if(af == AF_INET6){
1135
 
    ((struct sockaddr_in6 *)&to)->sin6_family = (sa_family_t)af;
1136
 
    ret = inet_pton(af, ip, &((struct sockaddr_in6 *)&to)->sin6_addr);
 
1135
    struct sockaddr_in6 *to6 = (struct sockaddr_in6 *)&to;
 
1136
    *to6 = (struct sockaddr_in6){ .sin6_family = (sa_family_t)af };
 
1137
    ret = inet_pton(af, ip, &to6->sin6_addr);
1137
1138
  } else {                      /* IPv4 */
1138
 
    ((struct sockaddr_in *)&to)->sin_family = (sa_family_t)af;
1139
 
    ret = inet_pton(af, ip, &((struct sockaddr_in *)&to)->sin_addr);
 
1139
    struct sockaddr_in *to4 = (struct sockaddr_in *)&to;
 
1140
    *to4 = (struct sockaddr_in){ .sin_family = (sa_family_t)af };
 
1141
    ret = inet_pton(af, ip, &to4->sin_addr);
1140
1142
  }
1141
1143
  if(ret < 0 ){
1142
1144
    int e = errno;
1221
1223
                    sizeof(struct sockaddr_in));
1222
1224
    }
1223
1225
    if(ret < 0){
1224
 
      if(errno == ENETUNREACH
 
1226
      if(((errno == ENETUNREACH) or (errno == EHOSTUNREACH))
1225
1227
         and if_index != AVAHI_IF_UNSPEC
1226
1228
         and connect_to == NULL
1227
1229
         and not route_added and
1465
1467
    free(decrypted_buffer);
1466
1468
    free(buffer);
1467
1469
    if(tcp_sd >= 0){
1468
 
      ret = (int)TEMP_FAILURE_RETRY(close(tcp_sd));
 
1470
      ret = close(tcp_sd);
1469
1471
    }
1470
1472
    if(ret == -1){
1471
1473
      if(e == 0){
1635
1637
    errno = ret_errno;
1636
1638
    return false;
1637
1639
  }
1638
 
  strcpy(ifr->ifr_name, ifname);
 
1640
  strncpy(ifr->ifr_name, ifname, IF_NAMESIZE);
 
1641
  ifr->ifr_name[IF_NAMESIZE-1] = '\0'; /* NUL terminate */
1639
1642
  ret = ioctl(s, SIOCGIFFLAGS, ifr);
1640
1643
  if(ret == -1){
1641
1644
    if(debug){
2010
2013
        perror_plus("openat");
2011
2014
        _exit(EXIT_FAILURE);
2012
2015
      }
2013
 
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
2016
      if(close(hookdir_fd) == -1){
2014
2017
        perror_plus("close");
2015
2018
        _exit(EXIT_FAILURE);
2016
2019
      }
2019
2022
        perror_plus("dup2(devnull, STDIN_FILENO)");
2020
2023
        _exit(EX_OSERR);
2021
2024
      }
2022
 
      ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
2025
      ret = close(devnull);
2023
2026
      if(ret == -1){
2024
2027
        perror_plus("close");
2025
2028
        _exit(EX_OSERR);
2074
2077
    free(direntry);
2075
2078
  }
2076
2079
  free(direntries);
2077
 
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
2080
  if(close(hookdir_fd) == -1){
2078
2081
    perror_plus("close");
2079
2082
  } else {
2080
2083
    hookdir_fd = -1;
2120
2123
    }
2121
2124
    
2122
2125
    if(quit_now){
2123
 
      ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2126
      ret = close(sd);
2124
2127
      if(ret == -1){
2125
2128
        perror_plus("close");
2126
2129
      }
2176
2179
    }
2177
2180
    
2178
2181
    /* Close the socket */
2179
 
    ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2182
    ret = close(sd);
2180
2183
    if(ret == -1){
2181
2184
      perror_plus("close");
2182
2185
    }
2264
2267
    }
2265
2268
    
2266
2269
    /* Close the socket */
2267
 
    int ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2270
    int ret = close(sd);
2268
2271
    if(ret == -1){
2269
2272
      perror_plus("close");
2270
2273
    }
2286
2289
 
2287
2290
int main(int argc, char *argv[]){
2288
2291
  mandos_context mc = { .server = NULL, .dh_bits = 0,
2289
 
                        .priority = "SECURE256:!CTYPE-X.509:"
2290
 
                        "+CTYPE-OPENPGP:!RSA", .current_server = NULL,
2291
 
                        .interfaces = NULL, .interfaces_size = 0 };
 
2292
                        .priority = "SECURE256:!CTYPE-X.509"
 
2293
                        ":+CTYPE-OPENPGP:!RSA:+SIGN-DSA-SHA256",
 
2294
                        .current_server = NULL, .interfaces = NULL,
 
2295
                        .interfaces_size = 0 };
2292
2296
  AvahiSServiceBrowser *sb = NULL;
2293
2297
  error_t ret_errno;
2294
2298
  int ret;
2521
2525
              }
2522
2526
            }
2523
2527
          }
2524
 
          TEMP_FAILURE_RETRY(close(seckey_fd));
 
2528
          close(seckey_fd);
2525
2529
        }
2526
2530
      }
2527
2531
      
2542
2546
              }
2543
2547
            }
2544
2548
          }
2545
 
          TEMP_FAILURE_RETRY(close(pubkey_fd));
 
2549
          close(pubkey_fd);
2546
2550
        }
2547
2551
      }
2548
2552
      
2549
 
      if(strcmp(dh_params_file, PATHDIR "/client-dhparams.pem" )
2550
 
         == 0){
 
2553
      if(dh_params_file != NULL
 
2554
         and strcmp(dh_params_file, PATHDIR "/dhparams.pem" ) == 0){
2551
2555
        int dhparams_fd = open(dh_params_file, O_RDONLY);
2552
2556
        if(dhparams_fd == -1){
2553
2557
          perror_plus("open");
2564
2568
              }
2565
2569
            }
2566
2570
          }
2567
 
          TEMP_FAILURE_RETRY(close(dhparams_fd));
 
2571
          close(dhparams_fd);
2568
2572
        }
2569
2573
      }
2570
2574
      
2747
2751
      errno = bring_up_interface(interface, delay);
2748
2752
      if(not interface_was_up){
2749
2753
        if(errno != 0){
2750
 
          perror_plus("Failed to bring up interface");
 
2754
          fprintf_plus(stderr, "Failed to bring up interface \"%s\":"
 
2755
                       " %s\n", interface, strerror(errno));
2751
2756
        } else {
2752
2757
          errno = argz_add(&interfaces_to_take_down,
2753
2758
                           &interfaces_to_take_down_size,
3082
3087
          perror_plus("rmdir");
3083
3088
        }
3084
3089
      }
3085
 
      TEMP_FAILURE_RETRY(close(tempdir_fd));
 
3090
      close(tempdir_fd);
3086
3091
    }
3087
3092
  }
3088
3093