talons

commit 84589f6e5c39b34633cf8985fcdab5845e381a69
parent b81ad835f10794501c11e3625c3a8a474443d342
Author: Olaf Hering <olaf@aepfle.de>
Date:   Wed, 22 Nov 2017 18:03:19 +0100

Move pointer usage after pointer check

In procmime_decode_content mimeinfo is dereferenced before cm_return_val_if_fail().

Diffstat:
M
src/procmime.c
 | 
4
++--

1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/procmime.c b/src/procmime.c
@@ -323,14 +323,14 @@ gboolean procmime_decode_content(MimeInfo *mimeinfo)
 	gint state = 0;
 	guint save = 0;
 
+	cm_return_val_if_fail(mimeinfo != NULL, FALSE);
+
 	EncodingType encoding = forced_encoding 
 				? forced_encoding
 				: mimeinfo->encoding_type;
 	gchar lastline[BUFFSIZE];
 	memset(lastline, 0, BUFFSIZE);
 
-	cm_return_val_if_fail(mimeinfo != NULL, FALSE);
-
 	if (prefs_common.respect_flowed_format &&
 	    mimeinfo->type == MIMETYPE_TEXT && 
 	    !strcasecmp(mimeinfo->subtype, "plain")) {