On Thu, Sep 27, 2012 at 4:21 PM, <shirishpargaonkar@xxxxxxxxx> wrote: > From: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> > > > Rebased and resending the reviewed patch. > Removed Reviewed by Jeff Layton since patch is slightly different > i.e. there is a change to smb1ops.c to cifs query_dir_first operation. > > Path based querries can fail for lack of access, especially during lookup > during open. > open itself would actually succeed becasue of back up intent bit > but querries (either path or file handle based) do not have a means to > specifiy backup intent bit. > So querry the file info during lookup using > trans2 / findfirst / file_id_full_dir_info > to obtain file info as well as file_id/inode value. Minor nitpick. I like the patch, but the commit message misspells "queries" a couple of times (sorry to be "that guy"). :) (sending to list, correctly formatted this time.) -- Peace and Blessings, -Scott. -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
