diff options
| author | Ben Chan <benchan@chromium.org> | 2018-10-17 23:06:47 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2018-10-18 13:17:36 +0200 |
| commit | 749b806c30b41512dec4901808976425cdf3e56b (patch) | |
| tree | 78cb67b2b673866583e3b17617adcfac30dbbc57 /src | |
| parent | 4156fd27186a517fcae938aaf23783000f6f34aa (diff) | |
core: remove unnecessary NULL checks for g_free()
g_free() already check if the given pointer is NULL and does nothing on
a NULL pointer.
Diffstat (limited to 'src')
| -rw-r--r-- | src/mm-base-sms.c | 6 | ||||
| -rw-r--r-- | src/mm-broadband-bearer.c | 3 |
2 files changed, 3 insertions, 6 deletions
diff --git a/src/mm-base-sms.c b/src/mm-base-sms.c index f9c16013..e117bef7 100644 --- a/src/mm-base-sms.c +++ b/src/mm-base-sms.c @@ -215,10 +215,8 @@ generate_3gpp_submit_pdus (MMBaseSms *self, } /* Free array (not contents, which were taken for the part) */ - if (split_text) - g_free (split_text); - if (split_data) - g_free (split_data); + g_free (split_text); + g_free (split_data); /* Set additional multipart specific properties */ if (n_parts > 1) { diff --git a/src/mm-broadband-bearer.c b/src/mm-broadband-bearer.c index bf02c13a..1b13921a 100644 --- a/src/mm-broadband-bearer.c +++ b/src/mm-broadband-bearer.c @@ -1376,8 +1376,7 @@ detailed_disconnect_finish (MMBroadbandBearer *self, static void detailed_disconnect_context_free (DetailedDisconnectContext *ctx) { - if (ctx->cgact_command) - g_free (ctx->cgact_command); + g_free (ctx->cgact_command); g_object_unref (ctx->data); g_object_unref (ctx->primary); if (ctx->secondary) |