git.99rst.org Git - openwrt-packages.git/blob

   1 commit bfa549da979e13f6c6a2e2defb7bbda5efa908f5
   2 Author: Jerome Magnin <jmagnin@haproxy.com>
   3 Date:   Fri Jan 17 16:09:33 2020 +0100
   4
   5     BUG/MINOR: pattern: handle errors from fgets when trying to load patterns
   6
   7     We need to do some error handling after we call fgets to make sure everything
   8     went fine. If we don't users can be fooled into thinking they can load pattens
   9     from directory because cfgparse doesn't flinch. This applies to acl patterns
  10     map files.
  11
  12     This should be backported to all supported versions.
  13
  14     (cherry picked from commit 3c79d4bdc47e151a97d7acdd99382bd9ca3927a5)
  15     Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
  16
  17 diff --git a/src/pattern.c b/src/pattern.c
  18 index ec7e9556a..33e0e17f2 100644
  19 --- a/src/pattern.c
  20 +++ b/src/pattern.c
  21 @@ -2328,6 +2328,11 @@ int pat_ref_read_from_file_smp(struct pat_ref *ref, const char *filename, char *
  22                 }
  23         }
  24
  25 +       if (ferror(file)) {
  26 +               memprintf(err, "error encountered while reading  <%s> : %s",
  27 +                               filename, strerror(errno));
  28 +               goto out_close;
  29 +       }
  30         /* succes */
  31         ret = 1;
  32
  33 @@ -2385,6 +2390,11 @@ int pat_ref_read_from_file(struct pat_ref *ref, const char *filename, char **err
  34                 }
  35         }
  36
  37 +       if (ferror(file)) {
  38 +               memprintf(err, "error encountered while reading  <%s> : %s",
  39 +                               filename, strerror(errno));
  40 +               goto out_close;
  41 +       }
  42         ret = 1; /* success */
  43
  44   out_close: