public inbox for lvm2-cvs@sourceware.org
help / color / mirror / Atom feed
* LVM2 ./WHATS_NEW lib/format_pool/import_export.c
@ 2010-12-01 13:05 zkabelac
  0 siblings, 0 replies; 2+ messages in thread
From: zkabelac @ 2010-12-01 13:05 UTC (permalink / raw)
  To: lvm-devel, lvm2-cvs

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	zkabelac@sourceware.org	2010-12-01 13:05:08

Modified files:
	.              : WHATS_NEW 
	lib/format_pool: import_export.c 

Log message:
	Check str_list_add() success
	
	Report error if str_list_add fails.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1825&r2=1.1826
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/format_pool/import_export.c.diff?cvsroot=lvm2&r1=1.32&r2=1.33

--- LVM2/WHATS_NEW	2010/12/01 13:01:36	1.1825
+++ LVM2/WHATS_NEW	2010/12/01 13:05:06	1.1826
@@ -1,5 +1,6 @@
 Version 2.02.78 - 
 ====================================
+  Check str_list_add() success in _add_stripe_seg().
   Check lv_info() success in _mirrored_transient_status().
   Add backtraces for dev_set() and dev_close_immediate() errors in set_lv().
   Add logging for unlink() error in clvmd remove_lockfile().
--- LVM2/lib/format_pool/import_export.c	2010/04/13 17:26:03	1.32
+++ LVM2/lib/format_pool/import_export.c	2010/12/01 13:05:07	1.33
@@ -206,7 +206,10 @@
 			return_0;
 
 	/* add the subpool type to the segment tag list */
-	str_list_add(mem, &seg->tags, _cvt_sptype(usp->type));
+	if (!str_list_add(mem, &seg->tags, _cvt_sptype(usp->type))) {
+		log_error("Allocation failed for str_list.");
+		return 0;
+	}
 
 	dm_list_add(&lv->segments, &seg->list);
 
@@ -240,7 +243,10 @@
 		}
 
 		/* add the subpool type to the segment tag list */
-		str_list_add(mem, &seg->tags, _cvt_sptype(usp->type));
+		if (!str_list_add(mem, &seg->tags, _cvt_sptype(usp->type))) {
+			log_error("Allocation failed for str_list.");
+			return 0;
+		}
 
 		if (!set_lv_segment_area_pv(seg, 0, usp->devs[j].pv, 0))
 			return_0;


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

* LVM2 ./WHATS_NEW lib/format_pool/import_export.c
@ 2005-07-26 21:48 alewis
  0 siblings, 0 replies; 2+ messages in thread
From: alewis @ 2005-07-26 21:48 UTC (permalink / raw)
  To: lvm2-cvs

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	alewis@sourceware.org	2005-07-26 21:48:18

Modified files:
	.              : WHATS_NEW 
	lib/format_pool: import_export.c 

Log message:
	fix pool format handler to work with pvseg code

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.268&r2=1.269
http://sources.redhat.com/cgi-bin/cvsweb.cgi/LVM2/lib/format_pool/import_export.c.diff?cvsroot=lvm2&r1=1.12&r2=1.13


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

end of thread, other threads:[~2010-12-01 13:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-01 13:05 LVM2 ./WHATS_NEW lib/format_pool/import_export.c zkabelac
  -- strict thread matches above, loose matches on Subject: below --
2005-07-26 21:48 alewis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).