From ffa73cbd5ca9dc306086deba95538d6dee89ec6e Mon Sep 17 00:00:00 2001
From: Jeffrey Stedfast <fejj@ximian.com>
Date: Fri, 4 Jan 2002 22:20:29 +0000
Subject: Don't try setting a content-type parameter if either the name or
 value is

2001-12-12  Jeffrey Stedfast  <fejj@ximian.com>

	* camel-folder-summary.c (content_info_load): Don't try setting a
	content-type parameter if either the name or value is NULL.

	* camel-mime-utils.c (header_set_param): NULL-protection.

svn path=/trunk/; revision=15248
---
 camel/camel-folder-summary.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'camel/camel-folder-summary.c')

diff --git a/camel/camel-folder-summary.c b/camel/camel-folder-summary.c
index 9a97a0bb59..d2b3c9fba6 100644
--- a/camel/camel-folder-summary.c
+++ b/camel/camel-folder-summary.c
@@ -1768,11 +1768,14 @@ content_info_load(CamelFolderSummary *s, FILE *in)
 	g_free(subtype);
 	if (camel_file_util_decode_uint32(in, &count) == -1 || count > 500)
 		goto error;
-	    
+	
 	for (i=0;i<count;i++) {
 		char *name, *value;
 		camel_folder_summary_decode_token(in, &name);
 		camel_folder_summary_decode_token(in, &value);
+		if (!(name && value))
+			goto error;
+		
 		header_content_type_set_param(ct, name, value);
 		/* TODO: do this so we dont have to double alloc/free */
 		g_free(name);
@@ -1791,7 +1794,7 @@ content_info_load(CamelFolderSummary *s, FILE *in)
 	if (!ferror(in))
 		return ci;
 
-error:
+ error:
 	camel_folder_summary_content_info_free(s, ci);
 	return NULL;
 }
-- 
cgit v1.2.3