diff options
| author | Sergey Poznyakoff <gray@gnu.org> | 2020-08-08 20:42:22 +0300 |
|---|---|---|
| committer | Sergey Poznyakoff <gray@gnu.org> | 2020-08-08 20:42:22 +0300 |
| commit | 1c8ed9b6b5e88a44e62ddbaa5978547a8670bbbe (patch) | |
| tree | 5308743948d53aa6f36bf92f082517f05891ed75 | |
| parent | b8553fee2f6da26406b7571e5dcfd2f4d604fe57 (diff) | |
| download | mailutils-1c8ed9b6b5e88a44e62ddbaa5978547a8670bbbe.tar.gz mailutils-1c8ed9b6b5e88a44e62ddbaa5978547a8670bbbe.tar.bz2 | |
Fix referencing uninitialized value
* decodemail/decodemail.c (message_decode_mime): Don't dereference
newmsg prior to creating it.
| -rw-r--r-- | decodemail/decodemail.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/decodemail/decodemail.c b/decodemail/decodemail.c index 1ec18d314..e75bdbf8c 100644 --- a/decodemail/decodemail.c +++ b/decodemail/decodemail.c @@ -716,10 +716,10 @@ message_decode_mime (mu_message_t msg, mu_coord_t *crd, size_t dim) rc = mu_message_get_part (msg, i, &msgpart); if (rc) { - mu_diag_funcall (MU_DIAG_ERROR, "mu_message_get_num_parts", + mu_diag_funcall (MU_DIAG_ERROR, "mu_message_get_part", NULL, rc); mu_mime_unref (mime); - goto end; + return NULL; } msgdec = message_decode (msgpart, crd, dim); rc = mu_mime_add_part (mime, msgdec); @@ -728,7 +728,7 @@ message_decode_mime (mu_message_t msg, mu_coord_t *crd, size_t dim) { mu_diag_funcall (MU_DIAG_ERROR, "mu_mime_add_part", NULL, rc); mu_mime_unref (mime); - goto end; + return NULL; } } |