On Tue, Jul 30, 2013 at 08:03:49AM +0800, Harald Welte wrote: > This seems to be a bogus warning. We do properly free the pdp library > handle, but we don't set pctx->lib to NULL in this path, see the below > suggested patch.
The warning is actually right.
gtp.c:
if (gsn->cb_conf)
gsn->cb_conf(qmsg->type, EOF, NULL, qmsg->cbp);
the NULL is the pdp context here
> + OSMO_ASSERT(pctx->lib == pdp);
did fail because of that. I will check and free pctx->lib as well and
push it in a couple of minutes.
