30 Jul
2013
30 Jul
'13
8:38 a.m.
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.