# HG changeset patch
# User Luke Schierer <lschiere@pidgin.im>
# Date 1045070176 0
# Node ID 3fef2d45dce046255920342b94621e5544ebdad0
# Parent  0ea6a4c53f38d1c51796696107a46780b658fd79
[gaim-migrate @ 4861]
(12:13:17) ari: This fixes a double-free, resulting in a crash when MALLOC_CHECK_=2, and fixes a problem where when the browser is set to Manual and you click on a link, the same link then becomes corrupted
(12:14:14) ari: basically the last one was because whoever changed browser.c assumed strdelimit returned a new string

committer: Tailor Script <tailor@pidgin.im>

diff -r 0ea6a4c53f38 -r 3fef2d45dce0 src/browser.c
--- a/src/browser.c	Wed Feb 12 04:53:21 2003 +0000
+++ b/src/browser.c	Wed Feb 12 17:16:16 2003 +0000
@@ -568,7 +568,8 @@
 				return;
 			}
 
-			space_free_url = g_strdelimit(url, " ", '+');
+			space_free_url = g_strdup(url);
+			g_strdelimit(space_free_url, " ", '+');
 			command = g_strdup_printf("%s %s", web_command, space_free_url);
  			g_free(space_free_url);
  			} break;
diff -r 0ea6a4c53f38 -r 3fef2d45dce0 src/module.c
--- a/src/module.c	Wed Feb 12 04:53:21 2003 +0000
+++ b/src/module.c	Wed Feb 12 17:16:16 2003 +0000
@@ -664,7 +664,8 @@
 			if (g_module_symbol(p->handle, "gaim_plugin_remove", (gpointer *)&gaim_plugin_remove))
 				gaim_plugin_remove();
 		}
-		g_free(p);
+		if(p)
+			g_free(p);
 		c = c->next;
 	}
 }