]> granicus.if.org Git - apache/commit
projects / apache / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 5e77c85) | patch
cppCheck: Suspicious condition.
authorChristophe Jaillet <jailletc36@apache.org>
Sat, 10 Nov 2012 17:21:42 +0000 (17:21 +0000)
committerChristophe Jaillet <jailletc36@apache.org>
Sat, 10 Nov 2012 17:21:42 +0000 (17:21 +0000)
commit9eb44f881890fc5b38108f808b0ce0f72e22e9bf
tree2bf6cff91eca75b615ebb55b9ebcf67a6b4b2286tree | snapshot
parent5e77c8547794e1cacbb0d99e68d53e8117c689f7commit | diff
cppCheck:  Suspicious condition.

AFAIK, there is no actual problem with the existing code.  'rv' would only not have the 'expected' value, but the execution
path would remain the same.
However, the patched code is more logical. We put the returned value in 'rv', then test it against 0.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1407853 13f79535-47bb-0310-9956-ffa450edef68
modules/aaa/mod_authz_dbd.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.