Project

General

Profile

Bug #1740

add unit tests for queue_seqdel() and callers

Added by neels almost 3 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
06/01/2016
Due date:
% Done:

0%

Spec Reference:

Description

See http://lists.osmocom.org/pipermail/osmocom-net-gprs/2016-May/000616.html
and following mails in this thread.

diff --git a/gtp/queue.c b/gtp/queue.c
index 5b4d849..fbfa1ec 100644
--- a/gtp/queue.c
+++ b/gtp/queue.c
@@ -105,8 +105,7 @@ static int queue_seqdel(struct queue_t *queue, struct qmsg_t *qmsg)
              printf("Begin queue_seqdel seq = %d\n", (int)qmsg->seq);

      for (qmsg2 = queue->hashseq[hash]; qmsg2; qmsg2 = qmsg2->seqnext) {
-             /* FIXME: this is always true !?! */
-             if (qmsg == qmsg) {
+             if (qmsg == qmsg2) {

queue_seqdel() is/was fundamentally broken.
We want unit tests for

  • queue_seqdel()
  • queue_freemsg()
  • queue_freemsg_seq()

as well as

  • gtp_conf()
  • gtp_retrans()

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)