On Thu, May 26, 2016 at 3:46 PM, Hillf Danton <hillf...@alibaba-inc.com> wrote: >> > See if this fixes your reproducer. >> > >> > diff --git a/fs/xattr.c b/fs/xattr.c >> > index b11945e..49b8eab 100644 >> > --- a/fs/xattr.c >> > +++ b/fs/xattr.c >> > @@ -667,6 +667,9 @@ xattr_resolve_name(const struct xattr_handler >> > **handlers, const char **name) >> > { >> > const struct xattr_handler *handler; >> > >> > + if (!handlers) >> > + return NULL; >> > + >> > if (!*name) >> > return NULL; >> > >> >> Tried, but doesn't work. >> > See if this fixes your reproducer. > > --- linux-4.6/fs/xattr.c Mon May 16 06:43:13 2016 > +++ b/fs/xattr.c Thu May 26 15:36:14 2016 > @@ -667,8 +667,8 @@ xattr_resolve_name(const struct xattr_ha > { > const struct xattr_handler *handler; > > - if (!*name) > - return NULL; > + if (!handlers || !*name) > + return ERR_PTR(-EINVAL); > > for_each_xattr_handler(handlers, handler) { > const char *n; > --
Hillf, That worked and it's already in Linus's latest tree. Thanks for sharing. Jeff