diff pidgin/gtkblist.c @ 24830:fbfbebc9197e

Fix two related bugs: 1. When renaming a group and only changing capitalization, Pidgin prompted you "You are about to merge "Some Group" and "some group" 2. When renaming a group and only changing capitalization, libpurple got into an infinite loop because the dest and source groups were the same, and it tried to add buddies to itself until itself was empty. This resulted in increasing memory and 100% CPU utilization until the process was killed. Did we recently change purple_find_group() to be case insensitive or something?
author Mark Doliner <mark@kingant.net>
date Mon, 22 Dec 2008 22:01:34 +0000
parents fdd9fd5c2086
children 289227f729ec 43b721aa4b76 45434c3fd878
line wrap: on
line diff
--- a/pidgin/gtkblist.c	Mon Dec 22 19:29:59 2008 +0000
+++ b/pidgin/gtkblist.c	Mon Dec 22 22:01:34 2008 +0000
@@ -602,7 +602,7 @@
 			break;
 		case PURPLE_BLIST_GROUP_NODE:
 			dest = purple_find_group(arg2);
-			if (dest != NULL && strcmp(arg2, ((PurpleGroup*) node)->name)) {
+			if (dest != NULL && purple_utf8_strcasecmp(arg2, ((PurpleGroup*) node)->name)) {
 				pidgin_dialogs_merge_groups((PurpleGroup*) node, arg2);
 			} else
 				purple_blist_rename_group((PurpleGroup*)node, arg2);