colin/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

modem-helpers: plug memleak

Browse Source 
g_match_info_fetch() returns always a heap-allocated string which should be
freed by the caller.
This commit is contained in:
Aleksander Morgado
2012-02-29 16:51:50 +01:00
parent 30e7063094
commit bc118aa160
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/mm-modem-helpers.c
View File

@@ -910,9 +910,10 @@ mm_gsm_parse_clck_test_response (const char *reply,
if (g_regex_match_full (r, p, strlen (p), 0, 0, &match_info, NULL)) {
while (g_match_info_matches (match_info)) {
str = g_match_info_fetch (match_info, 1);
if (str)
if (str) {
facilities |= mm_gsm_string_to_facility (str);
g_free (str);
g_free (str);
}
g_match_info_next (match_info, NULL);
success = TRUE;
@@ -956,6 +957,8 @@ mm_gsm_parse_clck_response (const char *reply, gboolean *enabled)
*enabled = TRUE;
else
success = FALSE;
g_free (str);
}
}
g_match_info_free (match_info);
@@ -1271,4 +1274,3 @@ done:
return array;
}