Re: [Acl-devel] libattr 2.4.48 change in syscalls.c breaks fakechroot

[Dmitry V. Levin]

On Mon, Aug 13, 2018 at 10:12:57PM -0700, Filipe Brandenburger wrote:
> This is not strictly a problem with libattr itself...
> 
> But when using LD_PRELOAD and dlsym() (like fakechroot does) the end
> result is an infinite loop that ends up crashing a program using one
> of the setxattr functions, due to the current implementation (trying
> to call the glibc functions) now binding the the stronger ones from
> the LD_PRELOAD library.
> 
> See here for more details:
> https://github.com/dex4er/fakechroot/issues/57

I suppose fakechroot has to be fixed to remain useful.

> This currently breaks `make test` in "rpm" package:
> https://github.com/rpm-software-management/rpm/issues/511

rpm test suite heavily relies on fakechroot which is inherently unreliable,
this has been a source of problems for quite a while.

> Any chance you'd consider going back to an implementation directly
> using SYSCALL(...) in libattr?

I believe commit v2.4.48~17 was the right change in this respect.
Note that software has been using glibc functions to access acl syscalls
for quite a long time, so changing libacl won't help fakechroot to handle
that software.


-- 
ldv