<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72"> <div class="WordSection1"> Hi,<o:p></o:p> <o:p> </o:p> I am using the libmm-glib API to query bearer status periodically and synchronously from MM(v1.6.4) from my own linux process. I have used mmcli as an example as was suggested on this forum previously.<o:p></o:p> (https://lists.freedesktop.org/archives/modemmanager-devel/2017-September/005700.html)<o:p></o:p> After few queries of bearer status, I see my process leaking memory. I think I am freeing all the allocations using g_object_unref and g_free but not sure what I am leaving behind.<o:p></o:p> I am not very familiar with glib memory management routines or with libmm-glib internals and so I am asking for help here hoping someone can give me some direction.<o:p></o:p> <o:p> </o:p> The following function is called periodically to get the bearer object(to read bearer connection status) and I have marked the object memory allocations and frees. I think I’m freeing all the allocations yet I see my process leaking memory due to calling this function.<o:p></o:p> Is there something obviously wrong? Is there a better way to query bearer connectivity status from MM?<o:p></o:p> <o:p> </o:p> <o:p> </o:p> func_get_bearer() {<o:p></o:p> <o:p> </o:p> /* ALLOC 1 */<o:p></o:p> ctx = g_new0(Context, 1); /* Context a struct which has MMManager MMObject, MMMOdem etc. */<o:p></o:p> <o:p> </o:p> /* ALLOC 2 */<o:p></o:p> mgr = mm_manager_new_sync(connection_object,<o:p></o:p> G_DBUS_OBJECT_MANAGER_CLIENT_FLAGS_DO_NOT_AUTO_START,<o:p></o:p> NULL, &error);<o:p></o:p> <o:p> </o:p> /* ALLOC 3 */<o:p></o:p> name = g_dbus_object_manager_client_get_name_owner(G_DBUS_OBJECT_MANAGER_CLIENT(mgr));<o:p></o:p> if(name == NULL)<o:p></o:p> {<o:p></o:p> g_object_unref(mgr);<o:p></o:p> /* Error */<o:p></o:p> return;<o:p></o:p> }<o:p></o:p> /* FREE 3 */<o:p></o:p> g_free(name); <o:p></o:p> <o:p> </o:p> /* Get modem path */<o:p></o:p> /* ALLOC 4 */<o:p></o:p> modem_path = g_strdup_printf(MM_DBUS_MODEM_PREFIX "/%s", modem_id); /* modem_id may be “0” */<o:p></o:p> <o:p> </o:p> /* Get modem object */<o:p></o:p> /* ALLOC 5 */<o:p></o:p> modems = g_dbus_object_manager_get_objects(G_DBUS_OBJECT_MANAGER(mgr));<o:p></o:p> for(mdm = modems; mdm; mdm = g_list_next(mdm))<o:p></o:p> {<o:p></o:p> mdm = MM_OBJECT(mdm->data);<o:p></o:p> if((modem_path != NULL) && g_str_equal(mm_object_get_path(mm_obj), modem_path))<o:p></o:p> {<o:p></o:p> /* ALLOC 6 */<o:p></o:p> found = g_object_ref(mm_obj);<o:p></o:p> break;<o:p></o:p> }<o:p></o:p> }<o:p></o:p> /* FREE 5 */<o:p></o:p> g_list_free_full(modems, g_object_unref);<o:p></o:p> /* FREE 4 */<o:p></o:p> g_free(modem_path);<o:p></o:p> <o:p> </o:p> ctx->m_generic_object = found;<o:p></o:p> <o:p> </o:p> /* ALLOC 7 */<o:p></o:p> ctx->mdm_intferface_obj = mm_object_get_modem(ctx->m_generic_object);<o:p></o:p> <o:p> </o:p> /* Get list of bearers */<o:p></o:p> /* ALLOC 8 */<o:p></o:p> b_list = mm_modem_list_bearers_sync(ctx->mdm_intferface_obj, NULL, &error);<o:p></o:p> <o:p> </o:p> /* Construct bearer path */<o:p></o:p> /* ALLOC 9 */<o:p></o:p> bearer_path = g_strdup_printf(MM_DBUS_BEARER_PREFIX "/%s", path_or_idx);<o:p></o:p> <o:p> </o:p> /* Get bearer object */<o:p></o:p> GList *bearer;<o:p></o:p> MMBearer *bearer_obj;<o:p></o:p> <o:p> </o:p> for(bearer = b_list; bearer; bearer = g_list_next(bearer))<o:p></o:p> {<o:p></o:p> bearer_obj = MM_BEARER(bearer->data);<o:p></o:p> if(bearer_obj == NULL)<o:p></o:p> {<o:p></o:p> continue;<o:p></o:p> }<o:p></o:p> if(g_str_equal(mm_bearer_get_path(bearer_obj), bearer_path))<o:p></o:p> {<o:p></o:p> /* Got bearer object */<o:p></o:p> /* ALLOC 10 */<o:p></o:p> ctx->bearer_obj = g_object_ref(bearer_obj);<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p> </o:p> /* Free the allocations */<o:p></o:p> /* FREE 8 */<o:p></o:p> g_list_free_full(b_list, g_object_unref);<o:p></o:p> <o:p> </o:p> /* FREE 10 */<o:p></o:p> g_object_unref(ctx->bearer_obj);<o:p></o:p> /* FREE 6 */<o:p></o:p> g_object_unref(ctx->m_generic_object);<o:p></o:p> /* FREE 7 */<o:p></o:p> g_object_unref(ctx->mdm_intferface_obj);<o:p></o:p> /* FREE 2 */<o:p></o:p> g_object_unref(ctx->mgr);<o:p></o:p> <o:p> </o:p> /* FREE 9 */<o:p></o:p> g_free(bearer_path);<o:p></o:p> /* FREE 1 */<o:p></o:p> g_free(ctx);<o:p></o:p> <o:p> </o:p> } /* End */<o:p></o:p> <o:p> </o:p> <o:p> </o:p> Thanks,<o:p></o:p> Shijo<o:p></o:p> </div> </body> </html>