30 Jul
2013
30 Jul
'13
6: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.