# HG changeset patch # User Mark Doliner # Date 1250965459 0 # Node ID 1d2425f2e4ff0cb5ffc761c74459128073530375 # Parent bf093003785bacb8fbd76a9e291d71a13496ec5d Woah, this was wrong. MY BAD! We should not be setting this to true just because we get an unknown html tag (or in this case an unescaped '<'). This fixes bug #10021 and I think probably another duplicate bug diff -r bf093003785b -r 1d2425f2e4ff libpurple/protocols/yahoo/util.c --- a/libpurple/protocols/yahoo/util.c Sat Aug 22 07:02:21 2009 +0000 +++ b/libpurple/protocols/yahoo/util.c Sat Aug 22 18:24:19 2009 +0000 @@ -611,7 +611,6 @@ if (match == NULL) { /* Unknown tag. The user probably typed a less-than sign */ g_string_append_c(cdata, x[i]); - no_more_gt_brackets = TRUE; g_free(tag); g_free(tag_name); break; diff -r bf093003785b -r 1d2425f2e4ff libpurple/tests/test_yahoo_util.c --- a/libpurple/tests/test_yahoo_util.c Sat Aug 22 07:02:21 2009 +0000 +++ b/libpurple/tests/test_yahoo_util.c Sat Aug 22 18:24:19 2009 +0000 @@ -100,6 +100,8 @@ yahoo_codes_to_html("test")); assert_string_equal_free("test", yahoo_codes_to_html("\x1B[35mtest")); + assert_string_equal_free(":<", + yahoo_codes_to_html(":<")); #endif /* !USE_CSS_FORMATTING */ } END_TEST