From a316f36dfc7cc4217bfce0ac6d23b47f90cf8fd9 Mon Sep 17 00:00:00 2001 From: Even Rouault Date: Thu, 10 Aug 2017 14:43:16 +0200 Subject: Fix crash on encoding if using opj_set_default_encoder_parameters() without defining tcp_numlayers --- src/bin/jp2/opj_compress.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/bin') diff --git a/src/bin/jp2/opj_compress.c b/src/bin/jp2/opj_compress.c index 1bbcebde..494b366a 100644 --- a/src/bin/jp2/opj_compress.c +++ b/src/bin/jp2/opj_compress.c @@ -1618,9 +1618,12 @@ static int parse_cmdline_encoder(int argc, char **argv, return 1; } /* mod fixed_quality */ + /* if no rate entered, lossless by default */ + /* Note: post v2.2.0, this is no longer necessary, but for released */ + /* versions at the time of writing, this is needed to avoid crashes */ if (parameters->tcp_numlayers == 0) { - parameters->tcp_rates[0] = 0; /* MOD antonin : losslessbug */ + parameters->tcp_rates[0] = 0; parameters->tcp_numlayers++; parameters->cp_disto_alloc = 1; } -- cgit v1.2.3