Fix some issues scanbuild found

Tue, 25 May 2021 02:03:21 -0500

author
Gary Kramlich <grim@reaperworld.com>
date
Tue, 25 May 2021 02:03:21 -0500
changeset 40889
00c77e22fb64
parent 40888
b19789161e29
child 40890
631788774f96

Fix some issues scanbuild found

Testing Done:
Ran scanbuild and verified all the issues were fixed

Reviewed at https://reviews.imfreedom.org/r/668/

libpurple/buddyicon.c file | annotate | diff | comparison | revisions
libpurple/protocols/silc/ops.c file | annotate | diff | comparison | revisions
pidgin/gtkblist.c file | annotate | diff | comparison | revisions
pidgin/gtkconv.c file | annotate | diff | comparison | revisions
--- a/libpurple/buddyicon.c	Tue May 25 00:06:32 2021 -0500
+++ b/libpurple/buddyicon.c	Tue May 25 02:03:21 2021 -0500
@@ -483,7 +483,7 @@
                                    const gchar *filename, GError **error)
 {
 	gconstpointer data;
-	size_t len;
+	size_t len = 0;
 
 	data = purple_buddy_icon_get_data(icon, &len);
 
--- a/libpurple/protocols/silc/ops.c	Tue May 25 00:06:32 2021 -0500
+++ b/libpurple/protocols/silc/ops.c	Tue May 25 02:03:21 2021 -0500
@@ -205,12 +205,14 @@
 		if (channel && !chat)
 			goto out;
 
-		img = purple_image_new_from_data(g_memdup2(data, data_len), data_len);
+		img = purple_image_new_from_data(data, data_len);
 		if (!img)
 			goto out;
 		img_id = purple_image_store_add_temporary(img);
-		if (!img_id)
+		if (!img_id) {
+			g_clear_object(&img);
 			goto out;
+		}
 
 		cflags |= PURPLE_MESSAGE_IMAGES | PURPLE_MESSAGE_RECV;
 		g_snprintf(tmp, sizeof(tmp), "<img src=\""
@@ -225,6 +227,8 @@
 				tmp, cflags, time(NULL));
 		}
 
+		g_clear_object(&img);
+
 		goto out;
 	}
 
--- a/pidgin/gtkblist.c	Tue May 25 00:06:32 2021 -0500
+++ b/pidgin/gtkblist.c	Tue May 25 02:03:21 2021 -0500
@@ -3272,7 +3272,7 @@
 		GList *cur = NULL;
 		PurpleProtocolChatEntry *pce;
 		char *name, *value;
-		PurpleChatConversation *conv;
+		PurpleChatConversation *conv = NULL;
 		PidginBlistNode *bnode = g_object_get_data(G_OBJECT(node), UI_DATA);
 
 		chat = (PurpleChat *)node;
--- a/pidgin/gtkconv.c	Tue May 25 00:06:32 2021 -0500
+++ b/pidgin/gtkconv.c	Tue May 25 02:03:21 2021 -0500
@@ -2717,7 +2717,6 @@
 	GtkTreePath *newpath;
 	const char *stock;
 	GtkTreeIter iter;
-	gboolean is_me = FALSE;
 	gboolean is_buddy;
 	const gchar *name, *alias;
 	gchar *tmp, *alias_key;
@@ -2741,9 +2740,6 @@
 
 	stock = get_chat_user_status_icon(chat, name, flags);
 
-	if (purple_strequal(purple_chat_conversation_get_nick(chat), purple_normalize(purple_conversation_get_account(conv), old_name != NULL ? old_name : name)))
-		is_me = TRUE;
-
 	is_buddy = purple_chat_user_is_buddy(cb);
 
 	tmp = g_utf8_casefold(alias, -1);

mercurial