Hi Ronald, > src/service.c | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > diff --git a/src/service.c b/src/service.c > index f148252..851db34 100644 > --- a/src/service.c > +++ b/src/service.c > @@ -984,7 +984,12 @@ static gboolean load_message_from_store(const char > *service_id, > if (mms_message_decode(pdu, len, msg) == FALSE) { > mms_error("Failed to decode %s", data_path); > munmap(pdu, len); > - goto out; > + g_free(state);
are we not leaking data_path here? > + > + mms_store_meta_close(service_id, uuid, meta, FALSE); > + mms_store_remove(service_id, uuid); > + > + return FALSE; > } And why are we turning a perfect good label into something were we have the label and also duplicated code. This is not a good idea. Regards Marcel _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono