Re: [Acl-devel] [PATCH] attr: refer to ENODATA instead of ENOATTR in man pages

[Florian Weimer]

On 03/27/2014 07:19 AM, Mike Frysinger wrote:
> On Tue 21 Jan 2014 14:27:33 Kamil Dudka wrote:
> > The ENOATTR alias is only provided if <attr/xattr.h> is included, not
> > with the more standard <sys/xattr.h> header file.  ENODATA is correct
> > in both cases.
>
> well, that's the case if the OS doesn't support ENOATTR, so the library uses
> ENODATA in its place.  if the OS does have its own, then we return ENOATTR
> everywhere, so checking ENODATA won't help.
>
> it might be the case on Linux that ENOATTR == ENODATA, but i think the goal is
> for libattr to be more than Linux.

My concern is that it's confusing on Linux that you don't always get 
ENOATTR, so if you write code based on this manual page, it does not work.

Hmm, perhaps we should just make sure that glibc header defines ENOATTR, 
too?  That would resolve this issues as well.

--
Florian Weimer / Red Hat Product Security Team