public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 3/9] dlm: don't free lvb twice
@ 2005-06-02  8:02 David Teigland
  0 siblings, 0 replies; only message in thread
From: David Teigland @ 2005-06-02  8:02 UTC (permalink / raw)
  To: akpm, linux-kernel

[-- Attachment #1: double-free-lvb.patch --]
[-- Type: text/plain, Size: 774 bytes --]

Don't free an rsb's lvb before calling free_rsb() because free_rsb() does
that.

Signed-off-by: David Teigland <teigland@redhat.com>

Index: linux/drivers/dlm/lockspace.c
===================================================================
--- linux.orig/drivers/dlm/lockspace.c	2005-06-02 12:28:30.000000000 +0800
+++ linux/drivers/dlm/lockspace.c	2005-06-02 12:55:58.290164152 +0800
@@ -460,10 +460,6 @@
 					 res_hashchain);
 
 			list_del(&rsb->res_hashchain);
-
-			if (rsb->res_lvbptr)
-				free_lvb(rsb->res_lvbptr);
-
 			free_rsb(rsb);
 		}
 
@@ -472,10 +468,6 @@
 			rsb = list_entry(head->next, struct dlm_rsb,
 					 res_hashchain);
 			list_del(&rsb->res_hashchain);
-
-			if (rsb->res_lvbptr)
-				free_lvb(rsb->res_lvbptr);
-
 			free_rsb(rsb);
 		}
 	}

--


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2005-06-02  8:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-06-02  8:02 [patch 3/9] dlm: don't free lvb twice David Teigland

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox