/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: 2016-08-06 00:53:13 UTC
  • Revision ID: teddy@recompile.se-20160806005313-q9n4b1b7707hnjj4
Makefile: Replace "-fsanitize=address" with "-fsanitize=leak"

The Address Sanitizer is a debugging feature, not a security feature -
it has security issues:  <http://seclists.org/oss-sec/2016/q1/363>
Therefore, it should only be used when debugging.  Replace it with
"-fsanitize=leak", which is needed since -fsanitize=address no longer
includes it implicitly.

* Makefile (DEBUG): Add "-fsanitize=address".
  (ALL_SANITIZE_OPTIONS): Replace "-fsanitize=address" with
                          "-fsanitize=leak".

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-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
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
 
      ret = close(dhpfile);
617
 
      if(ret == -1){
618
 
        perror_plus("close");
619
 
      }
620
615
      if(params.data == NULL){
621
616
        dhparamsfilename = NULL;
622
617
      }
631
626
                     safer_gnutls_strerror(ret));
632
627
        dhparamsfilename = NULL;
633
628
      }
634
 
      free(params.data);
635
629
    } while(false);
636
630
  }
637
631
  if(dhparamsfilename == NULL){
1084
1078
    bool match = false;
1085
1079
    {
1086
1080
      char *interface = NULL;
1087
 
      while((interface = argz_next(mc->interfaces,
1088
 
                                   mc->interfaces_size,
1089
 
                                   interface))){
 
1081
      while((interface=argz_next(mc->interfaces, mc->interfaces_size,
 
1082
                                 interface))){
1090
1083
        if(if_nametoindex(interface) == (unsigned int)if_index){
1091
1084
          match = true;
1092
1085
          break;
1499
1492
  return retval;
1500
1493
}
1501
1494
 
 
1495
__attribute__((nonnull))
1502
1496
static void resolve_callback(AvahiSServiceResolver *r,
1503
1497
                             AvahiIfIndex interface,
1504
1498
                             AvahiProtocol proto,
1659
1653
      perror_plus("ioctl SIOCGIFFLAGS");
1660
1654
      errno = old_errno;
1661
1655
    }
1662
 
    if((close(s) == -1) and debug){
1663
 
      old_errno = errno;
1664
 
      perror_plus("close");
1665
 
      errno = old_errno;
1666
 
    }
1667
1656
    return false;
1668
1657
  }
1669
 
  if((close(s) == -1) and debug){
1670
 
    old_errno = errno;
1671
 
    perror_plus("close");
1672
 
    errno = old_errno;
1673
 
  }
1674
1658
  return true;
1675
1659
}
1676
1660
 
1937
1921
      return;
1938
1922
    }
1939
1923
  }
1940
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1941
 
  if(devnull == -1){
1942
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1943
 
    return;
1944
 
  }
1945
1924
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1946
1925
                           runnable_hook, alphasort);
1947
1926
  if(numhooks == -1){
1948
1927
    perror_plus("scandir");
1949
 
    close(devnull);
1950
1928
    return;
1951
1929
  }
1952
1930
  struct dirent *direntry;
1953
1931
  int ret;
 
1932
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1933
  if(devnull == -1){
 
1934
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1935
    return;
 
1936
  }
1954
1937
  for(int i = 0; i < numhooks; i++){
1955
1938
    direntry = direntries[i];
1956
1939
    if(debug){
2212
2195
  
2213
2196
  /* Sleep checking until interface is running.
2214
2197
     Check every 0.25s, up to total time of delay */
2215
 
  for(int i = 0; i < delay * 4; i++){
 
2198
  for(int i=0; i < delay * 4; i++){
2216
2199
    if(interface_is_running(interface)){
2217
2200
      break;
2218
2201
    }
3039
3022
      /* Take down the network interfaces which were brought up */
3040
3023
      {
3041
3024
        char *interface = NULL;
3042
 
        while((interface = argz_next(interfaces_to_take_down,
3043
 
                                     interfaces_to_take_down_size,
3044
 
                                     interface))){
 
3025
        while((interface=argz_next(interfaces_to_take_down,
 
3026
                                   interfaces_to_take_down_size,
 
3027
                                   interface))){
3045
3028
          ret = take_down_interface(interface);
3046
3029
          if(ret != 0){
3047
3030
            errno = ret;
3076
3059
                                                | O_PATH));
3077
3060
    if(dir_fd == -1){
3078
3061
      perror_plus("open");
3079
 
      return;
3080
3062
    }
3081
3063
    int numentries = scandirat(dir_fd, ".", &direntries,
3082
3064
                               notdotentries, alphasort);
3099
3081
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3100
3082
            dret = 0;
3101
3083
          }
3102
 
          if((dret == -1) and (errno != ENOENT)){
 
3084
          if(dret == -1){
3103
3085
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3104
3086
                         direntries[i]->d_name, strerror(errno));
3105
3087
          }
3109
3091
      
3110
3092
      /* need to clean even if 0 because man page doesn't specify */
3111
3093
      free(direntries);
 
3094
      if(numentries == -1){
 
3095
        perror_plus("scandirat");
 
3096
      }
3112
3097
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3113
3098
      if(dret == -1 and errno != ENOENT){
3114
3099
        perror_plus("rmdir");