LIBOPENDKIM: Re-fix bug #226: Deal with header fields that are

wrapped before there's any content. Somehow the wrong fix was committed. Originally reported by Alessandro Vesely; re-reported by David Stevenson.
trusteddomainproject · Oct 7, 2015 · a654421 · a654421
1 parent 276d67b
commit a654421
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/RELEASE_NOTES b/RELEASE_NOTES
@@ -8,6 +8,10 @@ release, and a summary of the changes in that release.
 		by Iosif Fettich.
 	dkimf_db_nextpunct() could incorrectly identify an encoded hex digit
 		as a value delimiter.
+	LIBOPENDKIM: Re-fix bug #226: Deal with header fields that are
+		wrapped before there's any content.  Somehow the wrong fix
+		was committed.  Originally reported by Alessandro Vesely;
+		re-reported by David Stevenson.
 	LIBOPENDKIM: Fix bug #233: In the weird case of duplicate signatures,
 		constrain the size of the resulting "header.b" value.  Problem
 		noted by Joseph Coffland.

diff --git a/libopendkim/dkim-canon.c b/libopendkim/dkim-canon.c
@@ -388,7 +388,7 @@ dkim_canon_header_string(struct dkim_dstring *dstr, dkim_canon_t canon,
 		}
 
 		/* skip all spaces before first word */
-		while (*p != '\0' && DKIM_ISWSP(*p))
+		while (*p != '\0' && DKIM_ISLWSP(*p))
 			p++;
 
 		space = FALSE;				/* just saw a space */