Snapshot resizing memory leak

It looks like the current lvm code still has the memory leak when resizing snapshots. The lv_block_exception and lv_snapshot_hash_table fields are set to ones newly allocated for the new size, but the old ones are not freed. The following patch should fix it:

@@ -2489,6 +2489,8 @@

		memcpy(new_lv->lv_block_exception,
		       old_lv->lv_block_exception, size);

+ vfree(old_lv->lv_block_exception);

+		vfree(old_lv->lv_snapshot_hash_table);
		old_lv->lv_remap_end = new_lv->lv_remap_end;
		old_lv->lv_block_exception = new_lv->lv_block_exception;
		old_lv->lv_snapshot_hash_table =

Dale,

thanks for finding this one.
Could you please give the LVM version you patched in future posts ?

Regards,
Heinz -- The LVM Guy --

*** Software bugs are stupid.
Nevertheless it needs not so stupid people to solve them ***

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

Heinz Mauelshagen Sistina Software Inc.
Senior Consultant/Developer Am Sonnenhang 11
56242 Marienrachdorf Germany
[email protected] +49 2626 141200

                                                       FAX 924446

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-