Re: TLLI problems and proposed solution

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.