tech-pkg archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: LHA broken on modern Linux (was: Re: Changing the default .zoo extractor)
Am Sun, 2 Nov 2025 17:34:05 +0100
schrieb Jonathan Schleifer <js%pkgsrc.org@localhost>:
> Am 02.11.2025 um 17:22 schrieb Dr. Thomas Orgis <thomas.orgis%uni-hamburg.de@localhost>:
>
> > With that, it builds just fine with gcc 15.2.0 and should continue to
> > do so for future versions. Should I commit that?
>
> If you had a chance to test this and it works (e.g. `make extract` in cross/ppc-morphos-sdk`), please do!
>
It works (only extraction, not looked at more). I checked that really
gnu89 is needed (it is, because of // comments). There's a warning that
may bother some:
pm2tree.c: In function 'maketree2':
pm2tree.c:94:19: warning: writing 1 byte into a region of size 0 [-Wstringop-overflow=]
94 | [redacted, to not show non-free-licensed code]
| ~~~~~~~~~~^~~~~~~~~~~~
pm2tree.c:77:19: note: at offset 8 into destination object 'table2' of size 8
77 | [redacted]
| ^~~~~~
Looking at the code, I cannot quite see how the compiler decides this,
though. There's a loop with i < given_limit which is at most 8 in
calls, and in the loop body an assignment to table2[i]. Maybe I'm too
tired to see the overflow here.
Anyhow, I committed the FORCE_C_STD.
Alrighty then,
Thomas
--
Dr. Thomas Orgis
HPC @ Universität Hamburg
Home |
Main Index |
Thread Index |
Old Index