diff src/protocols/oscar/locate.c @ 7235:03f22e564311

[gaim-migrate @ 7810] I added some deps to the rpm, those should be there, right? Also... [ 821937 ] Fix for away message tooltip In Gaim 0.71, the tooltip text for Oscar is not properly escaped. This causes away messages such as "failed <-- this is a fact, not a prediction" to break the tooltip. Other protocols do not have this issue, since they call g_markup_escape_text() in the appropriate places. The call to gaim_markup_strip_html() is not sufficient in cases such as the example, since complete HTML tags are not present but HTML special characters are. This patch simply calls g_markup_escape_text() before adding the away message to the tooltip text. Patch is against the 0.71 source, since I don't think anoymous CVS is up-to-date. --Daniel Westermann-Clark (potpieman) committer: Tailor Script <[e-mail]>

--- a/src/protocols/oscar/locate.c	Sat Oct 11 01:58:59 2003 +0000
+++ b/src/protocols/oscar/locate.c	Sun Oct 12 03:15:38 2003 +0000
@@ -218,7 +218,7 @@
 		free(cur->info_encoding);
 		cur->info = (char *)malloc(userinfo->info_len);
 		memcpy(cur->info, userinfo->info, userinfo->info_len);
-		cur->info_encoding = strdup(userinfo->info_encoding); /* XXX - This seems to leak occasionally */
+		cur->info_encoding = strdup(userinfo->info_encoding);
 		cur->info_len = userinfo->info_len;
 	}
 
@@ -227,7 +227,7 @@
 		free(cur->away_encoding);
 		cur->away = (char *)malloc(userinfo->away_len);
 		memcpy(cur->away, userinfo->away, userinfo->away_len);
-		cur->away_encoding = strdup(userinfo->away_encoding); /* XXX - This seems to leak occasionally */
+		cur->away_encoding = strdup(userinfo->away_encoding);
 		cur->away_len = userinfo->away_len;
 	}
 }