diff options
author | Michael Scherer <misc@redhat.com> | 2017-02-23 20:17:25 +0100 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2017-02-28 23:29:38 -0500 |
commit | bae7b51d628a3ff4d96be1ec62dca1c8ed008dc8 (patch) | |
tree | 0ccba268cbfddba4d62ba0b87f3a0c4e080952f3 /README.md | |
parent | 3e66156cf202c56010bfa0c63db95ef0a5998947 (diff) |
Use int instead of int8_t for the 3 variables
Since strcmp return a int, and since the spec
of strcmp do not tell the return value, it
could return 256 and this would overflow.
Found by Coverity scan.
(thanks to Stéphane Marcheusin who explained
the details to me)
Change-Id: I5195e05b44f8b537226e6cee178d95a1ab904e96
BUG: 789278
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16738
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Michael Scherer <misc@fedoraproject.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'README.md')
0 files changed, 0 insertions, 0 deletions