diff options
author | NeilBrown <neilb@suse.de> | 2010-07-06 11:57:09 +1000 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2010-07-06 11:57:09 +1000 |
commit | e5c99c0811199a734ed0fe41784d22230922a49a (patch) | |
tree | 38312d403d91cfdc3f1ae20ff26139551a221497 | |
parent | b3b4e8a7a229cccca915421329a5319f996b0842 (diff) | |
download | mdadm-e5c99c0811199a734ed0fe41784d22230922a49a.tar.gz mdadm-e5c99c0811199a734ed0fe41784d22230922a49a.tar.xz mdadm-e5c99c0811199a734ed0fe41784d22230922a49a.zip |
Assemble: Fix honouring of 'auto' config line
commit 1ff98339283645a20c980d540f6c4d82693e7daf
broke the checking of metadata types via the 'auto' line.
Be moving 'load_super" before "conf_test_metadata" we left
tst->sb set even if conf_test_metadata fails, so the device will
actually be accepted and used.
So if we decide to reject the device, free the superblock so it is
clear that it is rejected.
Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r-- | Assemble.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -271,6 +271,7 @@ int Assemble(struct supertype *st, char *mddev, fprintf(stderr, Name ": %s has metadata type %s for which " "auto-assembly is disabled\n", devname, tst->ss->name); + tst->ss->free_super(tst); tmpdev->used = 2; } else { content = &info; |