[dm-devel] device-mapper ./WHATS_NEW dmeventd/libdevmappe ...

mornfall at sourceware.org mornfall at sourceware.org
Thu Aug 2 22:31:59 UTC 2007


CVSROOT:	/cvs/dm
Module name:	device-mapper
Changes by:	mornfall at sourceware.org	2007-08-02 22:31:59

Modified files:
	.              : WHATS_NEW 
	dmeventd       : libdevmapper-event.c 

Log message:
	Fix a possible segfault in libdevmapper-event. Thanks to Brian J. Wood
	for noticing this.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/device-mapper/WHATS_NEW.diff?cvsroot=dm&r1=1.196&r2=1.197
http://sourceware.org/cgi-bin/cvsweb.cgi/device-mapper/dmeventd/libdevmapper-event.c.diff?cvsroot=dm&r1=1.22&r2=1.23

--- device-mapper/WHATS_NEW	2007/07/28 10:48:36	1.196
+++ device-mapper/WHATS_NEW	2007/08/02 22:31:59	1.197
@@ -1,5 +1,6 @@
 Version 1.02.22 - 
 ================================
+  Fix a possible segfault in libdevmapper-event
   Export dm_create_dir (was create_dir) to help fix LVM2 link error
   Don't log mkdir fail-with-EROFS, to make create_dir equiv to the one in LVM2
   Introduce and use log_sys_* macros from LVM2
--- device-mapper/dmeventd/libdevmapper-event.c	2007/04/19 19:10:19	1.22
+++ device-mapper/dmeventd/libdevmapper-event.c	2007/08/02 22:31:59	1.23
@@ -706,6 +706,7 @@
 	}
 
 	dm_task_destroy(dmt);
+	dmt = 0;
 
 	if (msg.data) {
 		dm_free(msg.data);
@@ -759,7 +760,8 @@
 	if (reply_uuid)
 		dm_free(reply_uuid);
 	_dm_event_handler_clear_dev_info(dmevh);
-	dm_task_destroy(dmt);
+	if (dmt)
+		dm_task_destroy(dmt);
 	return ret;
 }
 




More information about the dm-devel mailing list