zstd icon indicating copy to clipboard operation
zstd copied to clipboard

[wip] Try to debug version test

Open terrelln opened this issue 3 years ago • 2 comments

No fix yet, just trying to debug what is going on.

terrelln avatar Dec 21 '22 00:12 terrelln

This is definitely a bug in v0.7.3. Though I don't quite know whats going on yet. I will look into it a little bit more, then try just skipping v0.7.3 in our tests.

terrelln avatar Dec 21 '22 01:12 terrelln

v0.7.3 could very well be buggy, but it's surprising that its associated test only starts to fail now. The test was expected to remain the same. What could have changed recently ?

Cyan4973 avatar Dec 21 '22 03:12 Cyan4973

On versions 0.7.3 - 0.8.0 (and possibly higher, I haven't tested yet), this line is failing:

https://github.com/facebook/zstd/blob/75ed1a815ef9695a2ba5afeb7f8e3f06d9c732a6/programs/fileio.c#L329

The error is: Operation not authorized at current processing stage

E.g. https://github.com/facebook/zstd/actions/runs/3752019678/jobs/6373711031

terrelln avatar Dec 21 '22 19:12 terrelln

ZBUFF_compressInit_advanced() could fail, for example, it could fail allocation.

That being said, the error message is more specific, so it should not be an allocation error.

Inside this function, ZSTD_compressBegin_advanced() is invoked, which itself invokes ZSTD_compress_insertDictionary(), which is active in the failing scenario. Yet, I fail to see any of these functions being able to trigger the stage_wrong error...

Cyan4973 avatar Dec 21 '22 19:12 Cyan4973

well it is randomly passing again, so I guess I'll close this PR

terrelln avatar Dec 21 '22 20:12 terrelln