Merge pull request #4464 from facebook/cli_traces_div0

fixed a potential division by 0 in the cli trace unit
This commit is contained in:
Yann Collet 2025-08-19 17:43:11 -07:00 committed by GitHub
commit 0551eaa851
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -81,8 +81,9 @@ static void TRACE_log(char const* method, PTime duration, ZSTD_Trace const* trac
{ {
int level = 0; int level = 0;
int workers = 0; int workers = 0;
double const durationAsDouble = duration ? (double)duration : .1;
double const ratio = (double)trace->uncompressedSize / (double)trace->compressedSize; double const ratio = (double)trace->uncompressedSize / (double)trace->compressedSize;
double const speed = ((double)trace->uncompressedSize * 1000) / (double)duration; double const speed = ((double)trace->uncompressedSize * 1000) / durationAsDouble;
if (trace->params) { if (trace->params) {
ZSTD_CCtxParams_getParameter(trace->params, ZSTD_c_compressionLevel, &level); ZSTD_CCtxParams_getParameter(trace->params, ZSTD_c_compressionLevel, &level);
ZSTD_CCtxParams_getParameter(trace->params, ZSTD_c_nbWorkers, &workers); ZSTD_CCtxParams_getParameter(trace->params, ZSTD_c_nbWorkers, &workers);