That's not really an error. It just would be smarter if xar_extract_tobuffersz() did something like this:

int err = xar_arcmod_extract(x,f,NULL,*buffer,*size); if (err != 0) { *size = 0; free(buffer); *buffer = NULL; } return err;

It would remove the need to to it in each caller.

Excellent suggestion, this has been resolved with commit 189.