/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: 2017-06-24 10:08:21 UTC
  • Revision ID: teddy@recompile.se-20170624100821-3ojepr4r6sifqzj5
Fix memory leak in password-prompt

* plugins.d/password-prompt.c (conflict_detection): Free any
  individual direntries, not just the direntry list.

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-2017 Teddy Hogeborn
 
13
 * Copyright © 2008-2017 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
 */
613
612
        }
614
613
        params.size += (unsigned int)bytes_read;
615
614
      }
616
 
      close(dhpfile);
617
615
      if(params.data == NULL){
618
616
        dhparamsfilename = NULL;
619
617
      }
1656
1654
      perror_plus("ioctl SIOCGIFFLAGS");
1657
1655
      errno = old_errno;
1658
1656
    }
1659
 
    close(s);
1660
1657
    return false;
1661
1658
  }
1662
 
  close(s);
1663
1659
  return true;
1664
1660
}
1665
1661
 
1926
1922
      return;
1927
1923
    }
1928
1924
  }
1929
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1930
 
  if(devnull == -1){
1931
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1932
 
    return;
1933
 
  }
1934
1925
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1935
1926
                           runnable_hook, alphasort);
1936
1927
  if(numhooks == -1){
1937
1928
    perror_plus("scandir");
1938
 
    close(devnull);
1939
1929
    return;
1940
1930
  }
1941
1931
  struct dirent *direntry;
1942
1932
  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
  }
1943
1938
  for(int i = 0; i < numhooks; i++){
1944
1939
    direntry = direntries[i];
1945
1940
    if(debug){
3065
3060
                                                | O_PATH));
3066
3061
    if(dir_fd == -1){
3067
3062
      perror_plus("open");
3068
 
      return;
3069
3063
    }
3070
3064
    int numentries = scandirat(dir_fd, ".", &direntries,
3071
3065
                               notdotentries, alphasort);
3088
3082
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3089
3083
            dret = 0;
3090
3084
          }
3091
 
          if((dret == -1) and (errno != ENOENT)){
 
3085
          if(dret == -1){
3092
3086
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3093
3087
                         direntries[i]->d_name, strerror(errno));
3094
3088
          }
3098
3092
      
3099
3093
      /* need to clean even if 0 because man page doesn't specify */
3100
3094
      free(direntries);
 
3095
      if(numentries == -1){
 
3096
        perror_plus("scandirat");
 
3097
      }
3101
3098
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3102
3099
      if(dret == -1 and errno != ENOENT){
3103
3100
        perror_plus("rmdir");