[PATCH 12/13] dlfcn,elf: implement dlmem() [BZ #11767]
Carlos O'Donell
carlos@redhat.com
Wed Mar 29 15:20:21 GMT 2023
On 3/29/23 10:50, stsp wrote:
>
> 29.03.2023 19:35, Carlos O'Donell пишет:
>> The fact that such binaries can be created is enough for me to raise
>> a sustained objection to the inclusion of dlmem().
>
> Then someone needs to tell how exactly
> it can be created, and, more importantly,
> point in the implementation the place
> that will then break.
It is not a reviewers task to show this to you, it is only their task that
they tell you that it exists. The onus is strongly on the patch submitter to
produce evidence to the contrary. The reviewers on the list here are a scarce
resource, and they offer the experience and technical knowledge to help you.
Again, I see no solution that fixes the design problems of dlmem(), there might
be a solution that includes two APIs that each do half the work of fdlopen().
Are you looking for coaching or mentoring in the design of system interfaces
for dynamic loading?
--
Cheers,
Carlos.
More information about the Libc-alpha
mailing list