[linux-cifs-client] [patch] Attempt #2 to handle null nameidata

[Jeff Layton]

On Thu, 29 Apr 2010 15:56:02 +0530
Suresh Jayaraman <sjayaraman at suse.de> wrote:

> On 04/09/2010 01:28 AM, Jeff Layton wrote:
> > On Thu, 8 Apr 2010 14:40:47 -0500
> > Shirish Pargaonkar <shirishpargaonkar at gmail.com> wrote:
> > 
> >> On Thu, Apr 8, 2010 at 2:34 PM, Jeff Layton <jlayton at samba.org> wrote:
> >>> On Wed, �7 Apr 2010 11:19:10 -0500
> >>> shirishpargaonkar at gmail.com wrote:
> >>>
> >>>> While creating a file on a server which supports unix extensions
> >>>> such as Samba, if a file is being created which does not supply
> >>>> nameidata (i.e. nd is null), cifs client can oops when calling
> >>>> cifs_posix_open.
> >>>>
> >>>> Signed-off-by: Shirish Pargaonkar <shirishpargaonkar at gmail.com>
> >>>> Reported-by: Eugene Teo <eugeneteo at kernel.sg>
> 
> > 
> > We'll need to take this patch in the interim though to fix the
> > immediate oops.
> > 
> 
> Do we need to Cc -stable as well as the issue seem to be reproducible on
> kernel versions > 2.6.29-rc6?
> 
> Thanks,
> 

Can someone clarify how to reproduce this oops? I think that the only
place where this function gets called with NULL nameidata is from nfsd
and the export ops for cifs are just stubs. Has this actually been seen
in the field or was it just found via inspection?

-- 
Jeff Layton <jlayton at samba.org>