Hi Giacinto, On 09/19/2018 11:42 PM, Giacinto Cifelli wrote: > --- > src/gprs.c | 4 ++++ > 1 file changed, 4 insertions(+) > diff --git a/src/gprs.c b/src/gprs.c > index 8f5d195..40f43e3 100644 > --- a/src/gprs.c > +++ b/src/gprs.c > @@ -1642,6 +1642,9 @@ static void release_active_contexts(struct ofono_gprs *gprs) > if (gc->driver->detach_shutdown != NULL) > gc->driver->detach_shutdown(gc, ctx->context.cid); > + > + /* Make sure the context is properly cleared */ > + release_context(ctx); > } > } > @@ -2241,6 +2244,7 @@ static DBusMessage *gprs_remove_context(DBusConnection *conn, > } > DBG("Unregistering context: %s", ctx->path); > + release_context(ctx); If we get here then the context isn't active and doesn't need to be released. Active context release happens in gprs_deactivate_for_remove. So I don't think this is needed?

> context_dbus_unregister(ctx); > gprs->contexts = g_slist_remove(gprs->contexts, ctx); Regards, -Denis