Giters

kspalaiologos / bzip3

A better and stronger spiritual successor to BZip2.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Addition of free(output) has hosed this for me

alerque opened this issue · comments

commented

I am not enough of an expert in memory management issues in C to suggest why, but merging #89 not only turned up failures across the board in CI (which could have been GitHub Actions having a bad moment) but it also fails for me locally when I build from Git HEAD:

$ make roundtrip
rm -f ./LICENSE2
./bzip3 -v -feb 6 ./LICENSE ./LICENSE.bz3
make: *** [Makefile:1206: roundtrip] Segmentation fault (core dumped)

It looks like the specific CI failures vary some by platform, but universally it either fails to build or fails to pass a round trip check (when it gets that far).