Actions
Bug #1740
closedadd unit tests for queue_seqdel() and callers
Start date:
06/01/2016
Due date:
% Done:
100%
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()
Related issues
Actions