Skip to content

Commit

Permalink
util/interval-tree: fix coverity warning
Browse files Browse the repository at this point in the history
Fix Coverity warning

** CID 1592992:  Incorrect expression  (COPY_PASTE_ERROR)
/src/util-port-interval-tree.c: 255 in SCPortIntervalFindOverlaps()

________________________________________________________________________________________________________
*** CID 1592992:  Incorrect expression  (COPY_PASTE_ERROR)
/src/util-port-interval-tree.c: 255 in SCPortIntervalFindOverlaps()
249                      * will be sorted, insert any new ports to the end of the list
250                      * and avoid walking the entire list */
251                     if (*list == NULL) {
252                         *list = new_port;
253                         (*list)->last = new_port;
254                     } else if (((*list)->last->port != new_port->port) &&
>>>     CID 1592992:  Incorrect expression  (COPY_PASTE_ERROR)
>>>     "port" in "(*list)->last->port2 != new_port->port" looks like a copy-paste error.
255                                ((*list)->last->port2 != new_port->port)) {
256                         DEBUG_VALIDATE_BUG_ON(new_port->port < (*list)->last->port);
257                         (*list)->last->next = new_port;
258                         new_port->prev = (*list)->last;
259                         (*list)->last = new_port;
260                     } else {

The code does not generate two port ranges that are same other than the
cases where port == port2 which is why it worked so far. Fix it.

Bug 6839
  • Loading branch information
inashivb authored and victorjulien committed Mar 8, 2024
1 parent af529a5 commit 2d6708f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/util-port-interval-tree.c
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ static void SCPortIntervalFindOverlaps(DetectEngineCtx *de_ctx, const uint16_t p
*list = new_port;
(*list)->last = new_port;
} else if (((*list)->last->port != new_port->port) &&
((*list)->last->port2 != new_port->port)) {
((*list)->last->port2 != new_port->port2)) {
DEBUG_VALIDATE_BUG_ON(new_port->port < (*list)->last->port);
(*list)->last->next = new_port;
new_port->prev = (*list)->last;
Expand Down

0 comments on commit 2d6708f

Please sign in to comment.