Merging my scanbuild/coverity fix

Tue, 10 May 2016 22:02:28 -0500

author
Gary Kramlich <grim@reaperworld.com>
date
Tue, 10 May 2016 22:02:28 -0500
changeset 37662
029eb4abd1b0
parent 37661
a2d2f84aef2c (current diff)
parent 37660
e0b280f4ab60 (diff)
child 37670
69a42dcf5a74

Merging my scanbuild/coverity fix

--- a/libpurple/http.c	Fri May 06 09:39:32 2016 -0500
+++ b/libpurple/http.c	Tue May 10 22:02:28 2016 -0500
@@ -2011,13 +2011,13 @@
 {
 	gchar *escaped_name = g_strdup(purple_url_encode(name));
 	gchar *escaped_value = NULL;
-	
-	if (escaped_value) {
+
+	if (value) {
 		escaped_value = g_strdup(purple_url_encode(value));
 	}
-	
+
 	purple_http_cookie_jar_set_ext(cookie_jar, escaped_name, escaped_value, -1);
-	
+
 	g_free(escaped_name);
 	g_free(escaped_value);
 }
--- a/libpurple/protocols/jabber/presence.c	Fri May 06 09:39:32 2016 -0500
+++ b/libpurple/protocols/jabber/presence.c	Tue May 10 22:02:28 2016 -0500
@@ -305,6 +305,8 @@
 	gboolean audio_enabled, video_enabled;
 #endif
 
+	g_return_val_if_fail(js !=NULL, NULL);
+
 	presence = purple_xmlnode_new("presence");
 
 	if(state == JABBER_BUDDY_STATE_UNAVAILABLE)
@@ -332,7 +334,7 @@
 	}
 
 	/* if we are idle and not offline, include idle */
-	if (js && js->idle && state != JABBER_BUDDY_STATE_UNAVAILABLE) {
+	if (js->idle && state != JABBER_BUDDY_STATE_UNAVAILABLE) {
 		PurpleXmlNode *query = purple_xmlnode_new_child(presence, "query");
 		gchar seconds[10];
 		g_snprintf(seconds, 10, "%d", (int) (time(NULL) - js->idle));
--- a/pidgin/gtkcertmgr.c	Fri May 06 09:39:32 2016 -0500
+++ b/pidgin/gtkcertmgr.c	Tue May 10 22:02:28 2016 -0500
@@ -317,7 +317,7 @@
 	GtkTreeModel *model;
 	gchar *id;
 	GTlsCertificate *crt;
-	char *title;
+	gchar *title;
 	GError *error = NULL;
 
 	/* See if things are selected */
@@ -331,7 +331,7 @@
 	gtk_tree_model_get(model, &iter, TPM_HOSTNAME_COLUMN, &id, -1);
 
 	/* Now retrieve the certificate */
-	crt = purple_tls_certificate_new_from_id(id, NULL);
+	crt = purple_tls_certificate_new_from_id(id, &error);
 
 	if (crt == NULL) {
 		purple_debug_warning("gtkcertmgr/tls_peers_mgmt",

mercurial