[PATCH weston v4 0/5] Add safe_strtoint() helper

Bryce Harrington bryce at osg.samsung.com
Thu Aug 4 00:40:47 UTC 2016


Incorporated a number of review suggestions:
  - Test against INT32_MIN instead of -INT_MAX
  - Don't modify *value on out of range conditions
  - A couple doc fixups
  - Add test cases accidentally missing in last post
  - Move test cases from patch #5 to #4

Bryce Harrington (5):
  Standardize error checking for strtol calls
  xwayland: Improve error checking for strtol call
  option-parser: Improve error checking for strtol call
  Add safe_strtoint() helper
  Switch to use safe_strtoint instead of strtol

 Makefile.am                 |  7 ++++
 compositor/main.c           |  6 ++--
 compositor/systemd-notify.c |  9 ++---
 libweston/compositor.c      |  8 ++---
 libweston/libbacklight.c    | 13 +++++--
 shared/config-parser.c      |  7 ++--
 shared/option-parser.c      | 12 +++++--
 shared/string-helpers.h     | 70 ++++++++++++++++++++++++++++++++++++
 tests/string-test.c         | 87 +++++++++++++++++++++++++++++++++++++++++++++
 xwayland/launcher.c         |  6 ++--
 10 files changed, 197 insertions(+), 28 deletions(-)
 create mode 100644 shared/string-helpers.h
 create mode 100644 tests/string-test.c

-- 
1.9.1



More information about the wayland-devel mailing list