diff options
| author | Guido Günther <agx@sigxcpu.org> | 2025-05-19 23:18:15 +0200 |
|---|---|---|
| committer | Dan Williams <dan@ioncontrol.co> | 2025-05-23 14:15:14 -0500 |
| commit | cdc6bf9529c37bff7197b479d3caa986bb9fc36b (patch) | |
| tree | 6f3bc86a001408317d8b67a8dfe91d26f7c172e4 /src/mm-cbm-part.c | |
| parent | 857efa1fba69cedaff48c1dff094c1b05afe66e8 (diff) | |
cbm-part: Use automatic cleanup
Safer than freeing the the struct in multiple places
Signed-off-by: Guido Günther <agx@sigxcpu.org>
Diffstat (limited to 'src/mm-cbm-part.c')
| -rw-r--r-- | src/mm-cbm-part.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/mm-cbm-part.c b/src/mm-cbm-part.c index 8ac92759..ef7a7807 100644 --- a/src/mm-cbm-part.c +++ b/src/mm-cbm-part.c @@ -74,7 +74,7 @@ mm_cbm_part_new_from_binary_pdu (const guint8 *pdu, gpointer log_object, GError **error) { - MMCbmPart *cbm_part; + g_autoptr (MMCbmPart) cbm_part = NULL; MMCbmGeoScope scope; guint offset = 0; guint16 serial, group; @@ -94,7 +94,6 @@ mm_cbm_part_new_from_binary_pdu (const guint8 *pdu, check_descr_str, \ pdu_len, \ required_size); \ - mm_cbm_part_free (cbm_part); \ return NULL; \ } @@ -116,7 +115,6 @@ mm_cbm_part_new_from_binary_pdu (const guint8 *pdu, mm_obj_dbg (log_object, " immediate cell cbm scope"); break; default: - mm_cbm_part_free (cbm_part); g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, @@ -155,7 +153,6 @@ mm_cbm_part_new_from_binary_pdu (const guint8 *pdu, else if (charset == CBS_DATA_CODING_GENERAL_UCS2) cbm_part->encoding = MM_SMS_ENCODING_UCS2; } else { - mm_cbm_part_free (cbm_part); g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED, @@ -190,12 +187,11 @@ mm_cbm_part_new_from_binary_pdu (const guint8 *pdu, log_object, error); if (!text) { - mm_cbm_part_free (cbm_part); return NULL; } cbm_part->text = g_steal_pointer (&text); - return cbm_part; + return g_steal_pointer (&cbm_part); } MMCbmPart * |