[Bug dynamic-link/30100] New: [patch] add dlmem()

["stsp at users dot sourceforge.net" <sourceware-bugzilla@sourceware.org>]

https://sourceware.org/bugzilla/show_bug.cgi?id=30100

            Bug ID: 30100
           Summary: [patch] add dlmem()
           Product: glibc
           Version: 2.38
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P2
         Component: dynamic-link
          Assignee: unassigned at sourceware dot org
          Reporter: stsp at users dot sourceforge.net
  Target Milestone: ---

Created attachment 14663
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14663&action=edit
fix bad free()

Hi guys.

I coded up the dlmem() support discussed here:
https://sourceware.org/bugzilla/show_bug.cgi?id=11767

Currently it consists of 2 patches: small fix to the
pre-existing bad free() problem, and dlmem() itself.
I'll attach them to this ticket.
Yes, I hear your screams "this patch sucks!". But in fact,
when you spend 99% of time thinking how to refactor this
1000+ lines of code function with multiple gotos, and only
1% of time goes to the coding itself, the patch is not going
to be pretty. :)
Also I can't be sure it doesn't introduce a regression.
After I added a member to struct rtld_global_ro, I had
to change the interpreter with patchelf on a tests I need
to run, because they are using the system-wide interpreter
where the struct rtld_global_ro is different. So I have
some tests failures now, but manually patching the interpreter
allows the test to succeed. So I can't properly validate
my patch with all tests.

I really wish someone from glibc to take over that patch. :)
glibc code is exceptionally alien to me, but the patch
works and has a test-case within.

-- 
You are receiving this mail because:
You are on the CC list for the bug.