Addition of free(output) has hosed this for me
alerque opened this issue · comments
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).