/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-07-20 00:06:05 UTC
  • mto: (237.7.594 trunk)
  • mto: This revision was merged to the branch mainline in revision 325.
  • Revision ID: teddy@recompile.se-20150720000605-244crrgqx254i3bs
Don't use TEMP_FAILURE_RETRY around close().

* plugin-runner.c: Don't use TEMP_FAILURE_RETRY around close().
* plugins.d/mandos-client.c: - '' -
* plugins.d/usplash.c: - '' -

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(), strcpy() */
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);
956
956
      perror_plus("openat");
957
957
      _exit(EX_UNAVAILABLE);
958
958
    }
959
 
    TEMP_FAILURE_RETRY(close(helperdir_fd));
 
959
    close(helperdir_fd);
960
960
#ifdef __GNUC__
961
961
#pragma GCC diagnostic push
962
962
#pragma GCC diagnostic ignored "-Wcast-qual"
1130
1130
    goto mandos_end;
1131
1131
  }
1132
1132
  
1133
 
  memset(&to, 0, sizeof(to));
1134
1133
  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);
 
1134
    struct sockaddr_in6 *to6 = (struct sockaddr_in6 *)&to;
 
1135
    *to6 = (struct sockaddr_in6){ .sin6_family = (sa_family_t)af };
 
1136
    ret = inet_pton(af, ip, &to6->sin6_addr);
1137
1137
  } 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);
 
1138
    struct sockaddr_in *to4 = (struct sockaddr_in *)&to;
 
1139
    *to4 = (struct sockaddr_in){ .sin_family = (sa_family_t)af };
 
1140
    ret = inet_pton(af, ip, &to4->sin_addr);
1140
1141
  }
1141
1142
  if(ret < 0 ){
1142
1143
    int e = errno;
1465
1466
    free(decrypted_buffer);
1466
1467
    free(buffer);
1467
1468
    if(tcp_sd >= 0){
1468
 
      ret = (int)TEMP_FAILURE_RETRY(close(tcp_sd));
 
1469
      ret = close(tcp_sd);
1469
1470
    }
1470
1471
    if(ret == -1){
1471
1472
      if(e == 0){
2010
2011
        perror_plus("openat");
2011
2012
        _exit(EXIT_FAILURE);
2012
2013
      }
2013
 
      if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
2014
      if(close(hookdir_fd) == -1){
2014
2015
        perror_plus("close");
2015
2016
        _exit(EXIT_FAILURE);
2016
2017
      }
2019
2020
        perror_plus("dup2(devnull, STDIN_FILENO)");
2020
2021
        _exit(EX_OSERR);
2021
2022
      }
2022
 
      ret = (int)TEMP_FAILURE_RETRY(close(devnull));
 
2023
      ret = close(devnull);
2023
2024
      if(ret == -1){
2024
2025
        perror_plus("close");
2025
2026
        _exit(EX_OSERR);
2074
2075
    free(direntry);
2075
2076
  }
2076
2077
  free(direntries);
2077
 
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
 
2078
  if(close(hookdir_fd) == -1){
2078
2079
    perror_plus("close");
2079
2080
  } else {
2080
2081
    hookdir_fd = -1;
2120
2121
    }
2121
2122
    
2122
2123
    if(quit_now){
2123
 
      ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2124
      ret = close(sd);
2124
2125
      if(ret == -1){
2125
2126
        perror_plus("close");
2126
2127
      }
2176
2177
    }
2177
2178
    
2178
2179
    /* Close the socket */
2179
 
    ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2180
    ret = close(sd);
2180
2181
    if(ret == -1){
2181
2182
      perror_plus("close");
2182
2183
    }
2264
2265
    }
2265
2266
    
2266
2267
    /* Close the socket */
2267
 
    int ret = (int)TEMP_FAILURE_RETRY(close(sd));
 
2268
    int ret = close(sd);
2268
2269
    if(ret == -1){
2269
2270
      perror_plus("close");
2270
2271
    }
2521
2522
              }
2522
2523
            }
2523
2524
          }
2524
 
          TEMP_FAILURE_RETRY(close(seckey_fd));
 
2525
          close(seckey_fd);
2525
2526
        }
2526
2527
      }
2527
2528
      
2542
2543
              }
2543
2544
            }
2544
2545
          }
2545
 
          TEMP_FAILURE_RETRY(close(pubkey_fd));
 
2546
          close(pubkey_fd);
2546
2547
        }
2547
2548
      }
2548
2549
      
2563
2564
              }
2564
2565
            }
2565
2566
          }
2566
 
          TEMP_FAILURE_RETRY(close(dhparams_fd));
 
2567
          close(dhparams_fd);
2567
2568
        }
2568
2569
      }
2569
2570
      
2746
2747
      errno = bring_up_interface(interface, delay);
2747
2748
      if(not interface_was_up){
2748
2749
        if(errno != 0){
2749
 
          perror_plus("Failed to bring up interface");
 
2750
          fprintf_plus(stderr, "Failed to bring up interface \"%s\":"
 
2751
                       " %s\n", interface, strerror(errno));
2750
2752
        } else {
2751
2753
          errno = argz_add(&interfaces_to_take_down,
2752
2754
                           &interfaces_to_take_down_size,
3081
3083
          perror_plus("rmdir");
3082
3084
        }
3083
3085
      }
3084
 
      TEMP_FAILURE_RETRY(close(tempdir_fd));
 
3086
      close(tempdir_fd);
3085
3087
    }
3086
3088
  }
3087
3089