[PATCH v2 0/3] Make sscanf() stricter

Demi Marie Obenour posted 3 patches 11 months, 1 week ago
Failed in applying to current master (apply log)
There is a newer version of this series
.../hive_isp_css_include/platform_support.h   |  1 -
drivers/xen/xenbus/xenbus_xs.c                | 17 ++--
include/linux/limits.h                        |  1 +
include/linux/mfd/wl1273-core.h               |  3 -
include/vdso/limits.h                         |  3 +
lib/vsprintf.c                                | 90 +++++++++++++------
6 files changed, 80 insertions(+), 35 deletions(-)
[PATCH v2 0/3] Make sscanf() stricter
Posted by Demi Marie Obenour 11 months, 1 week ago
Roger Pau Monné suggested making xenbus_scanf() stricter instead of
using a custom parser.  Christoph Hellwig asked why the normal vsscanf()
cannot be made stricter.  Richard Weinberger mentioned Linus Torvalds’s
suggestion of using ! to allow overflow.

Changes since v1:

- Better commit messages.
- Use ! to explicitly opt-in to allowing overflow.
- Treat overflow as a conversion failure instead of returning ERANGE.
- Drop the first patch (removal of simple_strtoll()) as it breaks
  bcache.
- Stop skipping spaces in vsscanf() instead of adding a separate
  vsscanf_strict() function.

Demi Marie Obenour (3):
  vsscanf(): Integer overflow is a conversion failure
  vsscanf(): do not skip spaces
  Strict XenStore entry parsing

 .../hive_isp_css_include/platform_support.h   |  1 -
 drivers/xen/xenbus/xenbus_xs.c                | 17 ++--
 include/linux/limits.h                        |  1 +
 include/linux/mfd/wl1273-core.h               |  3 -
 include/vdso/limits.h                         |  3 +
 lib/vsprintf.c                                | 90 +++++++++++++------
 6 files changed, 80 insertions(+), 35 deletions(-)

-- 
Sincerely,
Demi Marie Obenour (she/her/hers)
Invisible Things Lab