All of lore.kernel.org
 help / color / mirror / Atom feed
* [Patch 1/7] tabled: zero GList host_list or else crash
@ 2009-08-13  5:28 Pete Zaitcev
  2009-08-13  5:43 ` Jeff Garzik
  0 siblings, 1 reply; 2+ messages in thread
From: Pete Zaitcev @ 2009-08-13  5:28 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: Project Hail List

Awkward API: a Glib's list must be NULL at the start, and cldc_getaddr
appends to an existing list.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>

diff --git a/server/cldu.c b/server/cldu.c
index 9ce61b7..f1c2d4d 100644
--- a/server/cldu.c
+++ b/server/cldu.c
@@ -583,7 +583,7 @@ int cld_begin(const char *thishost, const char *thiscell,
 	}
 
 	if (!ses.forced_hosts) {
-		GList *tmp, *host_list;
+		GList *tmp, *host_list = NULL;
 		int i;
 
 		if (cldc_getaddr(&host_list, thishost, debugging, applog)) {

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Patch 1/7] tabled: zero GList host_list or else crash
  2009-08-13  5:28 [Patch 1/7] tabled: zero GList host_list or else crash Pete Zaitcev
@ 2009-08-13  5:43 ` Jeff Garzik
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2009-08-13  5:43 UTC (permalink / raw)
  To: Pete Zaitcev; +Cc: Project Hail List

Pete Zaitcev wrote:
> Awkward API: a Glib's list must be NULL at the start, and cldc_getaddr
> appends to an existing list.
> 
> Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>

applied 1-7



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-08-13  5:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-08-13  5:28 [Patch 1/7] tabled: zero GList host_list or else crash Pete Zaitcev
2009-08-13  5:43 ` Jeff Garzik

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.