changeset 27665:b7ce89597a89

ALLOW_TAG expands to multiple statements, so we should put braces around this if(). Otherwise, I think we'd accept <html/> in the middle of the markup. References #7340.
author Elliott Sales de Andrade <qulogic@pidgin.im>
date Sun, 19 Jul 2009 08:09:24 +0000
parents 989f1bd200b7
children 93e6cf770ea8
files libpurple/util.c
diffstat 1 files changed, 2 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/util.c	Sun Jul 19 06:20:21 2009 +0000
+++ b/libpurple/util.c	Sun Jul 19 08:09:24 2009 +0000
@@ -1572,8 +1572,9 @@
 				ALLOW_TAG("h5");
 				ALLOW_TAG("h6");
 				/* we only allow html to start the message */
-				if(c == html)
+				if(c == html) {
 					ALLOW_TAG("html");
+				}
 				ALLOW_TAG_ALT("i", "em");
 				ALLOW_TAG_ALT("italic", "em");
 				ALLOW_TAG("li");