/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: 2018-02-18 01:03:48 UTC
  • Revision ID: teddy@recompile.se-20180218010348-hhjfox5t4py5n5zh
Client: Bug fix: Possible memory leak if failing to open /dev/null

* plugins.d/mandos-client.d (run_network_hooks): Try to open /dev/null
  before reading network hook dir in order to fail early in that case.

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
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
 
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
21
23
 * WITHOUT ANY WARRANTY; without even the implied warranty of
22
24
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
23
25
 * General Public License for more details.
24
26
 * 
25
27
 * You should have received a copy of the GNU General Public License
26
 
 * along with this program.  If not, see
27
 
 * <http://www.gnu.org/licenses/>.
 
28
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
28
29
 * 
29
30
 * Contact the authors at <mandos@recompile.se>.
30
31
 */
1079
1080
    bool match = false;
1080
1081
    {
1081
1082
      char *interface = NULL;
1082
 
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
1083
 
                                 interface))){
 
1083
      while((interface = argz_next(mc->interfaces,
 
1084
                                   mc->interfaces_size,
 
1085
                                   interface))){
1084
1086
        if(if_nametoindex(interface) == (unsigned int)if_index){
1085
1087
          match = true;
1086
1088
          break;
1493
1495
  return retval;
1494
1496
}
1495
1497
 
1496
 
__attribute__((nonnull))
1497
1498
static void resolve_callback(AvahiSServiceResolver *r,
1498
1499
                             AvahiIfIndex interface,
1499
1500
                             AvahiProtocol proto,
1922
1923
      return;
1923
1924
    }
1924
1925
  }
 
1926
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1927
  if(devnull == -1){
 
1928
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1929
    return;
 
1930
  }
1925
1931
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1926
1932
                           runnable_hook, alphasort);
1927
1933
  if(numhooks == -1){
1928
1934
    perror_plus("scandir");
 
1935
    close(devnull);
1929
1936
    return;
1930
1937
  }
1931
1938
  struct dirent *direntry;
1932
1939
  int ret;
1933
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1934
 
  if(devnull == -1){
1935
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1936
 
    return;
1937
 
  }
1938
1940
  for(int i = 0; i < numhooks; i++){
1939
1941
    direntry = direntries[i];
1940
1942
    if(debug){
2196
2198
  
2197
2199
  /* Sleep checking until interface is running.
2198
2200
     Check every 0.25s, up to total time of delay */
2199
 
  for(int i=0; i < delay * 4; i++){
 
2201
  for(int i = 0; i < delay * 4; i++){
2200
2202
    if(interface_is_running(interface)){
2201
2203
      break;
2202
2204
    }
3023
3025
      /* Take down the network interfaces which were brought up */
3024
3026
      {
3025
3027
        char *interface = NULL;
3026
 
        while((interface=argz_next(interfaces_to_take_down,
3027
 
                                   interfaces_to_take_down_size,
3028
 
                                   interface))){
 
3028
        while((interface = argz_next(interfaces_to_take_down,
 
3029
                                     interfaces_to_take_down_size,
 
3030
                                     interface))){
3029
3031
          ret = take_down_interface(interface);
3030
3032
          if(ret != 0){
3031
3033
            errno = ret;
3082
3084
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3083
3085
            dret = 0;
3084
3086
          }
3085
 
          if(dret == -1){
 
3087
          if((dret == -1) and (errno != ENOENT)){
3086
3088
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3087
3089
                         direntries[i]->d_name, strerror(errno));
3088
3090
          }