diff options
| author | Ben Chan <benchan@chromium.org> | 2017-07-12 01:03:38 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2017-07-12 11:04:09 +0200 |
| commit | 39ddd34898d458d1105ac6fa168166b23229ab8e (patch) | |
| tree | 2b78ba723a159ae26074933e360dcd40cde65922 | |
| parent | 4a7190f64b86b47076e91b6ebfeb59e27baa2707 (diff) | |
telit: remove unnecessary NULL checks for g_free / g_match_info_free
g_free() and g_match_info_free() already check if the given pointer is
NULL and does nothing on a NULL pointer.
| -rw-r--r-- | plugins/telit/mm-modem-helpers-telit.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/plugins/telit/mm-modem-helpers-telit.c b/plugins/telit/mm-modem-helpers-telit.c index d0e50ab3..75667fd0 100644 --- a/plugins/telit/mm-modem-helpers-telit.c +++ b/plugins/telit/mm-modem-helpers-telit.c @@ -308,9 +308,7 @@ end: if (!ret && bands != NULL) g_array_free (bands, TRUE); - if(match_info) - g_match_info_free (match_info); - + g_match_info_free (match_info); g_regex_unref (r); return ret; @@ -361,8 +359,7 @@ mm_telit_get_2g_mm_bands (GMatchInfo *match_info, } end: - if (match_str != NULL) - g_free (match_str); + g_free (match_str); if (flags != NULL) g_array_free (flags, TRUE); @@ -432,8 +429,7 @@ mm_telit_get_3g_mm_bands (GMatchInfo *match_info, } end: - if (match_str != NULL) - g_free (match_str); + g_free (match_str); if (flags != NULL) g_array_free (flags, TRUE); @@ -486,8 +482,7 @@ mm_telit_get_4g_mm_bands(GMatchInfo *match_info, value = value >> 1; } end: - if (match_str != NULL) - g_free (match_str); + g_free (match_str); return ret; } |