diff options
| author | Joel Becker <joel.becker@oracle.com> | 2008-12-11 15:04:14 -0800 |
|---|---|---|
| committer | Mark Fasheh <mfasheh@suse.com> | 2009-01-05 08:40:34 -0800 |
| commit | d030cc978e9e636dc39ce9a9e8282d48698a3b30 (patch) | |
| tree | 0d9053e2968eba3505c8333b96869b9d2a16cccc | |
| parent | c175a518b4a1d514483abf61813ce5d855917164 (diff) | |
| download | linux-3.10-d030cc978e9e636dc39ce9a9e8282d48698a3b30.tar.gz linux-3.10-d030cc978e9e636dc39ce9a9e8282d48698a3b30.tar.bz2 linux-3.10-d030cc978e9e636dc39ce9a9e8282d48698a3b30.zip | |
ocfs2: Validate superblock with checksum and ecc.
The superblock is read via a raw call. Validate it after we find it
from its signature.
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
| -rw-r--r-- | fs/ocfs2/super.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c index 2eb657c3e7a..43ed11345b5 100644 --- a/fs/ocfs2/super.c +++ b/fs/ocfs2/super.c @@ -52,6 +52,7 @@ #include "ocfs1_fs_compat.h" #include "alloc.h" +#include "blockcheck.h" #include "dlmglue.h" #include "export.h" #include "extent_map.h" @@ -1989,6 +1990,15 @@ static int ocfs2_verify_volume(struct ocfs2_dinode *di, if (memcmp(di->i_signature, OCFS2_SUPER_BLOCK_SIGNATURE, strlen(OCFS2_SUPER_BLOCK_SIGNATURE)) == 0) { + /* We have to do a raw check of the feature here */ + if (le32_to_cpu(di->id2.i_super.s_feature_incompat) & + OCFS2_FEATURE_INCOMPAT_META_ECC) { + status = ocfs2_block_check_validate(bh->b_data, + bh->b_size, + &di->i_check); + if (status) + goto out; + } status = -EINVAL; if ((1 << le32_to_cpu(di->id2.i_super.s_blocksize_bits)) != blksz) { mlog(ML_ERROR, "found superblock with incorrect block " @@ -2030,6 +2040,7 @@ static int ocfs2_verify_volume(struct ocfs2_dinode *di, } } +out: mlog_exit(status); return status; } |