kernel.git/fs/cifs/dir.c, branch linux-3.6.y

cifs: Do not lookup hashed negative dentry in cifs_atomic_open

2012-11-26T20:14:07Z

commit 3798f47aa276b332c30da499cb4df4577e2f8872 upstream. We do not need to lookup a hashed negative directory since we have already revalidated it before and have found it to be fine. This also prevents a crash in cifs_lookup() when it attempts to rehash the already hashed negative lookup dentry. The patch has been tested using the reproducer at https://bugzilla.redhat.com/show_bug.cgi?id=867344#c28 Reported-by: Vit Zahradka Signed-off-by: Sachin Prabhu Signed-off-by: Greg Kroah-Hartman

CIFS: Fix cifs_do_create error hadnling

2012-08-20T03:30:18Z

Commit d2c127197dfc0b2bae62a52e1e0d3e3ff493919e caused a regression in cifs_do_create error handling. Fix this by closing a file handle in the case of a get_inode_info(_unix) error. Also remove unnecessary checks for newinode being NULL. Signed-off-by: Pavel Shilovsky Signed-off-by: Steve French

CIFS: Make CAP_* checks protocol independent

2012-07-24T19:12:03Z

Since both CIFS and SMB2 use ses->capabilities (server->capabilities) field but flags are different we should make such checks protocol independent. Reviewed-by: Jeff Layton Signed-off-by: Pavel Shilovsky Signed-off-by: Steve French

CIFS: Fix a wrong pointer in atomic_open

2012-07-24T17:54:26Z

Commit 30d904947459cca2beb69e0110716f5248b31f2a caused a regression in cifs open codepath. Reviewed-by: Jeff Layton Signed-off-by: Pavel Shilovsky Signed-off-by: Steve French

CIFS: Rename Get/FreeXid and make them work with unsigned int

2012-07-24T15:25:08Z

Acked-by: Jeff Layton Signed-off-by: Pavel Shilovsky Signed-off-by: Steve French

don't pass nameidata to ->create()

2012-07-14T12:34:47Z

boolean "does it have to be exclusive?" flag is passed instead; Local filesystem should just ignore it - the object is guaranteed not to be there yet. Signed-off-by: Al Viro

stop passing nameidata to ->lookup()

2012-07-14T12:34:32Z

Just the flags; only NFS cares even about that, but there are legitimate uses for such argument. And getting rid of that completely would require splitting ->lookup() into a couple of methods (at least), so let's leave that alone for now... Signed-off-by: Al Viro

stop passing nameidata * to ->d_revalidate()

2012-07-14T12:34:14Z

Just the lookup flags. Die, bastard, die... Signed-off-by: Al Viro

make finish_no_open() return int

2012-07-14T12:33:45Z

namely, 1 ;-) That's what we want to return from ->atomic_open() instances after finish_no_open(). Signed-off-by: Al Viro

kill struct opendata

2012-07-14T12:33:39Z

Just pass struct file *. Methods are happier that way... There's no need to return struct file * from finish_open() now, so let it return int. Next: saner prototypes for parts in namei.c Signed-off-by: Al Viro