[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Fri, 11 Dec 2009 16:06:58 +0200
From: Surbhi Palande <surbhi.palande@...onical.com>
To: linux-ext4@...r.kernel.org
Subject: [PATCH] replaced BUG() with return -EIO from ext4_ext_get_blocks
This patch fixes the upstream bug# 14286. When the address of an extent
corresponding to a given block is NULL and the tree is being traversed for
fetching such an address, a -EIO should be reported instead of a BUG(). This
situation should normally not occur. However if it does, then the system
should be rendered usable.
Signed-off-by: Surbhi Palande <surbhi.palande@...onical.com>
---
fs/ext4/extents.c | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
index 3a7928f..51f87f3 100644
--- a/fs/ext4/extents.c
+++ b/fs/ext4/extents.c
@@ -3190,7 +3190,12 @@ int ext4_ext_get_blocks(handle_t *handle, struct inode *inode,
* this situation is possible, though, _during_ tree modification;
* this is why assert can't be put in ext4_ext_find_extent()
*/
- BUG_ON(path[depth].p_ext == NULL && depth != 0);
+ if (path[depth].p_ext == NULL && depth != 0) {
+ err = -EIO;
+ printk(KERN_ERR "\n ext4 fs error in %s,%s,%s while reading a block ", \
+ __FILE__, __func__, __LINE__);
+ goto out2;
+ }
eh = path[depth].p_hdr;
ex = path[depth].p_ext;
--
1.6.3.3
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists