diff --git a/tests/bigdict.c b/tests/bigdict.c index 748b60e79..33c4ab8ea 100644 --- a/tests/bigdict.c +++ b/tests/bigdict.c @@ -70,39 +70,38 @@ int main(int argc, const char** argv) char* buffer = (char*)malloc(bufferSize); void* out = malloc(outSize); void* roundtrip = malloc(dataSize); - int _exit_code = 0; + int _exit_code = 1; (void)argc; (void)argv; if (!buffer || !out || !roundtrip || !cctx || !dctx) { fprintf(stderr, "Allocation failure\n"); - _exit_code = 1; goto cleanup; } if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_windowLog, 31))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_nbWorkers, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_overlapLog, 9))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_checksumFlag, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_strategy, ZSTD_btopt))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_targetLength, 7))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_minMatch, 7))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_searchLog, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_hashLog, 10))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_chainLog, 10))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_DCtx_setParameter(dctx, ZSTD_d_windowLogMax, 31))) - return 1; + goto cleanup; RDG_genBuffer(buffer, bufferSize, 1.0, 0.0, 0xbeefcafe); @@ -112,16 +111,16 @@ int main(int argc, const char** argv) for (i = 0; i < 10; ++i) { fprintf(stderr, "Compressing 1 GB\n"); if (compress(cctx, dctx, out, outSize, buffer, dataSize, roundtrip, ZSTD_e_continue)) - return 1; + goto cleanup; } } fprintf(stderr, "Compressing 1 GB\n"); if (compress(cctx, dctx, out, outSize, buffer, dataSize, roundtrip, ZSTD_e_end)) - return 1; + goto cleanup; fprintf(stderr, "Success!\n"); - goto cleanup; + _exit_code = 0; cleanup: free(roundtrip); diff --git a/tests/largeDictionary.c b/tests/largeDictionary.c index ff2bb2d70..998fd9fa8 100644 --- a/tests/largeDictionary.c +++ b/tests/largeDictionary.c @@ -70,37 +70,38 @@ int main(int argc, const char** argv) char* buffer = (char*)malloc(bufferSize); void* out = malloc(outSize); void* roundtrip = malloc(dataSize); + int _exit_code = 1; (void)argc; (void)argv; if (!buffer || !out || !roundtrip || !cctx || !dctx) { fprintf(stderr, "Allocation failure\n"); - return 1; + goto cleanup; } if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_windowLog, 31))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_nbWorkers, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_overlapLog, 9))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_checksumFlag, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_strategy, ZSTD_btopt))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_targetLength, 7))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_minMatch, 7))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_searchLog, 1))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_hashLog, 10))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_CCtx_setParameter(cctx, ZSTD_c_chainLog, 10))) - return 1; + goto cleanup; if (ZSTD_isError(ZSTD_DCtx_setParameter(dctx, ZSTD_d_windowLogMax, 31))) - return 1; + goto cleanup; RDG_genBuffer(buffer, bufferSize, 1.0, 0.0, 0xbeefcafe); @@ -110,19 +111,20 @@ int main(int argc, const char** argv) for (i = 0; i < 10; ++i) { fprintf(stderr, "Compressing 1 GB\n"); if (compress(cctx, dctx, out, outSize, buffer, dataSize, roundtrip, ZSTD_e_continue)) - return 1; + goto cleanup; } } fprintf(stderr, "Compressing 1 GB\n"); if (compress(cctx, dctx, out, outSize, buffer, dataSize, roundtrip, ZSTD_e_end)) - return 1; + goto cleanup; fprintf(stderr, "Success!\n"); +cleanup: free(roundtrip); free(out); free(buffer); ZSTD_freeDCtx(dctx); ZSTD_freeCCtx(cctx); - return 0; + return _exit_code; } diff --git a/tests/regression/method.c b/tests/regression/method.c index f84a15ef3..ac6fdd712 100644 --- a/tests/regression/method.c +++ b/tests/regression/method.c @@ -159,15 +159,15 @@ static result_t compress_cctx_compress( return result_error(result_error_skip); int const level = config_get_level(config); - + result_t result; ZSTD_CCtx* cctx = ZSTD_createCCtx(); ZSTD_DCtx* dctx = ZSTD_createDCtx(); if (cctx == NULL || dctx == NULL) { fprintf(stderr, "context creation failed\n"); - return result_error(result_error_system_error); + result = result_error(result_error_system_error); + goto out; } - result_t result; result_data_t data = {.total_size = 0}; for (size_t i = 0; i < state->inputs.size; ++i) { data_buffer_t const input = state->inputs.buffers[i];