summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorDan Bastone <dan@pwienterprises.com>2006-08-13 23:24:18 -0700
committerGreg Kroah-Hartman <gregkh@suse.de>2006-08-14 12:54:28 -0700
commit95f8797f42b058333d1e6f0d1dcd8edf5dc6c244 (patch)
treef0f0cc374b0d0d884210f3cdbd0774efe3b79fe9 /fs
parent1725cd0ae07bb31f68803edcc5bdc99952c7d2f4 (diff)
downloadop-kernel-dev-95f8797f42b058333d1e6f0d1dcd8edf5dc6c244.zip
op-kernel-dev-95f8797f42b058333d1e6f0d1dcd8edf5dc6c244.tar.gz
[PATCH] initialize parts of udf inode earlier in create
Eric says: > I saw an oops down this path when trying to create a new file on a UDF > filesystem which was internally marked as readonly, but mounted rw: > > udf_create > udf_new_inode > new_inode > alloc_inode > udf_alloc_inode > udf_new_block > returns EIO due to readonlyness > iput (on error) I ran into the same issue today, but when listing a directory with invalid/corrupt entries: udf_lookup udf_iget get_new_inode_fast alloc_inode udf_alloc_inode __udf_read_inode fails for any reason iput (on error) ... The following patch to udf_alloc_inode() should take care of both (and other similar) cases, but I've only tested it with udf_lookup(). Signed-off-by: Dan Bastone <dan@pwienterprises.com> Cc: Eric Sandeen <sandeen@sandeen.net> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
-rw-r--r--fs/udf/super.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/udf/super.c b/fs/udf/super.c
index 4df822c..7de172e 100644
--- a/fs/udf/super.c
+++ b/fs/udf/super.c
@@ -115,6 +115,13 @@ static struct inode *udf_alloc_inode(struct super_block *sb)
ei = (struct udf_inode_info *)kmem_cache_alloc(udf_inode_cachep, SLAB_KERNEL);
if (!ei)
return NULL;
+
+ ei->i_unique = 0;
+ ei->i_lenExtents = 0;
+ ei->i_next_alloc_block = 0;
+ ei->i_next_alloc_goal = 0;
+ ei->i_strat4096 = 0;
+
return &ei->vfs_inode;
}
OpenPOWER on IntegriCloud