Mercurial > pidgin3 / changeset

--- a/libpurple/protocols/gg/lib/dcc7.c	Thu Apr 21 09:40:21 2016 +0200
+++ b/libpurple/protocols/gg/lib/dcc7.c	Thu May 05 23:13:36 2016 -0500
@@ -333,6 +333,9 @@
 {
 	gg_debug_dcc(dcc, GG_DEBUG_FUNCTION, "** gg_dcc7_reverse_connect(%p)\n", dcc);
 
+	if (dcc == NULL)
+		return -1;
+
 	if (dcc->reverse) {
 		gg_debug_dcc(dcc, GG_DEBUG_MISC, "// gg_dcc7_reverse_connect() already reverse connection\n");
 		return -1;

--- a/libpurple/protocols/jabber/jabber.c	Thu Apr 21 09:40:21 2016 +0200
+++ b/libpurple/protocols/jabber/jabber.c	Thu May 05 23:13:36 2016 -0500
@@ -220,12 +220,13 @@
 static gboolean
 jabber_process_starttls(JabberStream *js, PurpleXmlNode *packet)
 {
+	PurpleXmlNode *starttls;
+
+#if 0
 	PurpleAccount *account;
-	PurpleXmlNode *starttls;
 
 	account = purple_connection_get_account(js->gc);
 
-#if 0
 	/*
 	 * This code DOES NOT EXIST, will never be enabled by default, and
 	 * will never ever be supported (by me).
@@ -245,6 +246,7 @@
 	return TRUE;
 #endif
 
+#if 0
 	starttls = purple_xmlnode_get_child(packet, "starttls");
 	if(purple_xmlnode_get_child(starttls, "required")) {
 		purple_connection_error(js->gc,
@@ -261,6 +263,7 @@
 	}
 
 	return FALSE;
+#endif
 }
 
 void jabber_stream_features_parse(JabberStream *js, PurpleXmlNode *packet)

--- a/libpurple/protocols/jabber/jingle/transport.c	Thu Apr 21 09:40:21 2016 +0200
+++ b/libpurple/protocols/jabber/jingle/transport.c	Thu May 05 23:13:36 2016 -0500
@@ -91,7 +91,7 @@
 static void
 jingle_transport_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec)
 {
-	g_return_if_fail(object != NULL);
+	g_return_if_fail(G_IS_OBJECT(object));
 	g_return_if_fail(JINGLE_IS_TRANSPORT(object));
 
 	switch (prop_id) {
@@ -104,7 +104,7 @@
 static void
 jingle_transport_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
 {
-	g_return_if_fail(object != NULL);
+	g_return_if_fail(G_IS_OBJECT(object));
 	g_return_if_fail(JINGLE_IS_TRANSPORT(object));
 
 	switch (prop_id) {

--- a/libpurple/tls-certificate-info.c	Thu Apr 21 09:40:21 2016 +0200
+++ b/libpurple/tls-certificate-info.c	Thu May 05 23:13:36 2016 -0500
@@ -302,7 +302,7 @@
 {
 	GSList *list;
 	GSList *ret = NULL;
-	DerOIDValue *value;
+	DerOIDValue *value = NULL;
 
 	g_return_val_if_fail(name_node != NULL, NULL);
 
@@ -382,6 +382,8 @@
 	g_return_val_if_fail(node != NULL, NULL);
 	g_return_val_if_fail(node->content != NULL, NULL);
 
+	memset(time_parts, 0, sizeof(gint) * sizeof(time_parts));
+
 	time = der_parse_string(node);
 
 	/* For the purposes of X.509

--- a/pidgin/gtkcertmgr.c	Thu Apr 21 09:40:21 2016 +0200
+++ b/pidgin/gtkcertmgr.c	Thu May 05 23:13:36 2016 -0500
@@ -336,7 +336,7 @@
 	if (crt == NULL) {
 		purple_debug_warning("gtkcertmgr/tls_peers_mgmt",
 				"Unable to fetch certificate '%s': %s",
-				id, error->message);
+				id, error ? error->message : "unknown error");
 		g_clear_error(&error);
 		g_free(id);
 	}