[gaim-migrate @ 16050]

Mon, 17 Apr 2006 23:53:40 +0000

author
Mark Doliner <markdoliner@pidgin.im>
date
Mon, 17 Apr 2006 23:53:40 +0000
changeset 13650
8c1043337393
parent 13649
b156749bec5b
child 13651
48f7048f1ec1

[gaim-migrate @ 16050]
This should fix Coverity CID 49: buddy was sometimes
leaking in _resolve_reply() in src/protocols/bonjour/dns_sd.c

src/protocols/bonjour/buddy.c file | annotate | diff | comparison | revisions
src/protocols/bonjour/buddy.h file | annotate | diff | comparison | revisions
src/protocols/bonjour/dns_sd.c file | annotate | diff | comparison | revisions
--- a/src/protocols/bonjour/buddy.c	Mon Apr 17 19:28:15 2006 +0000
+++ b/src/protocols/bonjour/buddy.c	Mon Apr 17 23:53:40 2006 +0000
@@ -27,9 +27,10 @@
  * Creates a new buddy.
  */
 BonjourBuddy *
-bonjour_buddy_new(gchar *name, gchar *first, gint port_p2pj,
-	gchar *phsh, gchar *status, gchar *email, gchar *last, gchar *jid, gchar *AIM,
-	gchar *vc, gchar *ip, gchar *msg)
+bonjour_buddy_new(const gchar *name, const gchar *first, gint port_p2pj,
+	const gchar *phsh, const gchar *status, const gchar *email,
+	const gchar *last, const gchar *jid, const gchar *AIM,
+	const gchar *vc, const gchar *ip, const gchar *msg)
 {
 	BonjourBuddy *buddy = malloc(sizeof(BonjourBuddy));
 
--- a/src/protocols/bonjour/buddy.h	Mon Apr 17 19:28:15 2006 +0000
+++ b/src/protocols/bonjour/buddy.h	Mon Apr 17 23:53:40 2006 +0000
@@ -43,9 +43,10 @@
 /**
  * Creates a new buddy.
  */
-BonjourBuddy *bonjour_buddy_new(gchar *name, gchar *first, gint port_p2pj,
-	gchar *phsh, gchar *status, gchar *email, gchar *last, gchar *jid, gchar *AIM,
-	gchar *vc, gchar *ip, gchar *msg);
+BonjourBuddy *bonjour_buddy_new(const gchar *name, const gchar *first,
+	gint port_p2pj, const gchar *phsh, const gchar *status,
+	const gchar *email, const gchar *last, const gchar *jid,
+	const gchar *AIM, const gchar *vc, const gchar *ip, const gchar *msg);
 
 /**
  * Check if all the compulsory buddy data is present.
--- a/src/protocols/bonjour/dns_sd.c	Mon Apr 17 19:28:15 2006 +0000
+++ b/src/protocols/bonjour/dns_sd.c	Mon Apr 17 23:53:40 2006 +0000
@@ -143,11 +143,12 @@
 
 	/* Put the parameters of the text_record in a buddy and add the buddy to */
 	/* the buddy list */
-	buddy = bonjour_buddy_new((gchar *)name, first, port_p2pj, phsh,
+	buddy = bonjour_buddy_new(name, first, port_p2pj, phsh,
 							  status, email, last, jid, AIM, vc, ip, msg);
 
 	if (bonjour_buddy_check(buddy) == FALSE)
 	{
+		bonjour_buddy_delete(buddy);
 		return SW_DISCOVERY_E_UNKNOWN;
 	}
 

mercurial