Hi,

On Mon, Jan 09, 2006 at 12:42:35PM -0500, Jeff Mahoney wrote:
>  Since heartbeat threads are stopped when the individual heartbeat sets are
>  removed, this just cleans up the allocation.
Yes, it seems we're leaking memory there.

> +     o2hb_free_hb_set(cluster->cl_group.default_groups[1]);
>       for (i = 0; cluster->cl_group.default_groups[i]; i++) {
>               killme = &cluster->cl_group.default_groups[i]->cg_item;
>               cluster->cl_group.default_groups[i] = NULL;
What about the for loop on the next line? Won't that try to access memory
freed in the call to o2hb_free_hb_set(...)?                 
        --Mark

--
Mark Fasheh
Senior Software Developer, Oracle
[EMAIL PROTECTED]
_______________________________________________
Ocfs2-devel mailing list
[email protected]
http://oss.oracle.com/mailman/listinfo/ocfs2-devel

Reply via email to