On Wed, Feb 21, 2007 at 06:07:56PM +0100, Jan Engelhardt wrote:
> Hello Greg,
>
>
> On Feb 20 2007 20:05, Greg KH wrote:
> >
> >Try this instead:
> > if (!de)
> > return -ENOMEM;
> > if ((IS_ERR(de)) && (PTR_ERR(de) != -ENODEV))
> > return PTR_ERR(de);
> > return 0;
> >
> >That should cover everything properly, right?
>
> In case memory could not be allocated, why does not securityfs_*() return
> ERR_PTR(-ENOMEM) then? (I think, that's the quintessential question after
> all. And thanks for giving an example what to do in the ENODEV case.)
Actually, in reading the code (which might have helped in the first
place), we can never return NULL if securityfs is enabled. So you can
just drop that first check entirely.
Which makes me wonder, it might be easier to just return NULL if
securityfs is not enabled in the kernel, as long as no one checks that
improperly...
Hope this helps,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
