diff options
author | Corey Farrell <git@cfware.com> | 2017-10-05 21:23:31 -0400 |
---|---|---|
committer | Corey Farrell <git@cfware.com> | 2017-10-05 20:28:19 -0500 |
commit | 3421b1de27da1229bd41cc5a65083231e4b60881 (patch) | |
tree | b546e8acd0a65336da68d2c775a18b94de90438e | |
parent | 0991cb0a8fc99a356683f4efa701ac21bfdc46ac (diff) |
main/strings: Fix uninitialized value.
ast_strings_match uses sscanf and checks for non-zero return to verify a
token was parsed. This is incorrect as sscanf returns EOF (-1) for errors.
ASTERISK-27318 #close
Change-Id: Ifcece92605f58116eff24c5a0a3b0ee08b3c87b1
-rw-r--r-- | main/strings.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/strings.c b/main/strings.c index 19aa2d373..3a333239c 100644 --- a/main/strings.c +++ b/main/strings.c @@ -314,7 +314,7 @@ regex: } equals: - scan_numeric = (sscanf(left, "%lf", &left_num) && sscanf(internal_right, "%lf", &right_num)); + scan_numeric = (sscanf(left, "%lf", &left_num) > 0 && sscanf(internal_right, "%lf", &right_num) > 0); if (internal_op[0] == '=') { if (ast_strlen_zero(left) && ast_strlen_zero(internal_right)) { |