search

NixOS / patchelf

A small utility to modify the dynamic linker and RPATH of ELF executables
GNU General Public License v3.0
3.57k stars 487 forks source link

Other switches might set changed as true. Use extraStrings size. #416 #473

Closed brenoguim closed 1 year ago

brenoguim commented 1 year ago

I didn't realize that "changed" could be set by other switches before reaching the renaming function. In this case, it tried to add zero strings and failed an assertion due to that. Added a test that breaks without the fix.

Mic92 commented 1 year ago

bors merge

bors[bot] commented 1 year ago

Build succeeded: