From 39ddd34898d458d1105ac6fa168166b23229ab8e Mon Sep 17 00:00:00 2001 From: Ben Chan Date: Wed, 12 Jul 2017 01:03:38 -0700 Subject: [PATCH] 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. --- plugins/telit/mm-modem-helpers-telit.c | 13 ++++--------- 1 file 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; }