upstream/haproxy
1
0
Fork 0
mirror of https://github.com/haproxy/haproxy.git synced 2026-03-22 10:30:34 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

CLEANUP: queue: Remove useless tests on p or pp in pendconn_process_next_strm()

Browse source 
This patch removes unecessary tests on p or pp pointers in
pendconn_process_next_strm() function. This should make cppcheck happy and
avoid false report of null pointer dereference.

This patch should fix the issue #1036.
This commit is contained in:
Christopher Faulet 2021-02-11 11:13:33 +01:00
parent a1e0f387c7
commit cd7126b396
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/queue.c
View file

@ -281,12 +281,12 @@ static int pendconn_process_next_strm(struct server *srv, struct proxy *px)
if (!p && !pp)
return 0;
else if (!pp)
goto use_p; /* p != NULL */
else if (!p)
goto use_pp; /* pp != NULL */
if (p && !pp)
goto use_p;
if (pp && !p)
goto use_pp;
/* p != NULL && pp != NULL*/
if (KEY_CLASS(p->node.key) < KEY_CLASS(pp->node.key))
goto use_p;