Neil Brown: Avoid possible null dereference in Assemble.
    Martin F. Krafft 
    madduck at alioth.debian.org
       
    Tue Jun 24 22:02:11 UTC 2008
    
    
  
Module: mdadm
Branch: master
Commit: ddd1a492519d61587cdd92660b875b82f4832176
URL:    http://git.debian.org/?p=pkg-mdadm/mdadm.git;a=commit;h=ddd1a492519d61587cdd92660b875b82f4832176
Author: Neil Brown <neilb at suse.de>
Date:   Thu May 15 15:50:42 2008 +1000
Avoid possible null dereference in Assemble.
If the first device we look at has no superblock,
there is no 'st' to free, so don't free it.
---
 Assemble.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/Assemble.c b/Assemble.c
index c7cc385..8621203 100644
--- a/Assemble.c
+++ b/Assemble.c
@@ -315,7 +315,8 @@ int Assemble(struct supertype *st, char *mddev, int mdfd,
 		if (!tst || !tst->sb) {
 			fprintf(stderr, Name ": %s has no superblock - assembly aborted\n",
 				devname);
-			st->ss->free_super(st);
+			if (st)
+				st->ss->free_super(st);
 			return 1;
 		}
 
    
    
More information about the pkg-mdadm-commits
mailing list