CMake: error out on warnings for strict/missing prototypes.

And fix strict-prototypes/missing-prototypes warnings.

Merge pull request #1461 from rouault/fix_ci

Fix CI

Fix CI

Merge pull request #1460 from rouault/fix_1459

opj_t2_skip_packet_data(): avoid out-of-bounds reads on truncated images in non-strict mode (fixes #1459)

opj_t2_skip_packet_data(): avoid out-of-bounds reads on truncated images in non-strict mode (fixes #1459)

CMake: add missing GNUInstallDirs related substitutions (fixes #1424) (#1456)

Merge pull request #1448 from rouault/fix_1447

Fix incorrect decoding of image with large number of progression levels

Fix incorrect decoding of image with large number of progression levels

Fixes regression introduced per d27ccf01c68a31ad62b33d2dc1ba2bb1eeaafe7b

Fixes #1447

Fix Heap-buffer-overflow READ in opj_jp2_apply_pclr (#1441)

The issue was found while fuzzing opencv:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=47342

The read overflow triggered by reading `src[j]` in
```cpp
            for (j = 0; j < max; ++j) {
                dst[j] = src[j];
            }
```
The max is calculated as `new_comps[pcol].w * new_comps[pcol].h`, however the `src = old_comps[cmp].data;` which may have different `w` and `h` dimensions.

Merge pull request #1440 from rouault/rate_alloc_speedup

Significant speed-up rate allocation by rate/distoratio ratio

Replace the assert in mel_init to an if statement to address an issue with fuzzing. (#1436)

Modified the mel_init code to replace the assert statement with an if statement, returning false when an incorrect sequence of bytes are encountered in the MEL segment.  Similar code should be added to the main MEL decoding subrountine, but the change is more involved; in any case, an incorrect sequence produces incorrect results, but should not be harmful or cause a crash.

Cleanup code related to quality layer allocation, and add a few safety checks

Significant speed-up rate allocation by rate/distoratio ratio

- Avoid doing 128 iterations all the time, and stop when the threshold
  doesn't vary much
- Avoid calling costly opj_t2_encode_packets() repeatdly when bisecting the
  layer ratio if the truncation points haven't changed since the last
  iteration.

When used with the GDAL gdal_translate application to convert a 11977 x
8745 raster with data type UInt16 and 8 channels, the conversion time
to JPEG2000 with 20 quality layers using disto/rate allocation (
-co "IC=C8" -co "JPEG2000_DRIVER=JP2OPENJPEG" -co "PROFILE=NPJE_NUMERICALLY_LOSSLESS"
creation options of the GDAL NITF driver) goes from 5m56 wall clock
(8m20s total, 12 vCPUs) down to 1m16 wall clock (3m45 total).

Micro-optimization: use directly opj_bio_putbit() instead of opj_bio_write() to emit single bit

Merge pull request #1439 from arichardson/relocatable-cmake-config

Make OpenJPEGConfig.cmake relocatable with CMake > 3.0

Make OpenJPEGConfig.cmake relocatable with CMake > 3.0

Using CMakePackageConfigHelpers, we can generate a relocatable
OpenJPEGConfig.config, using the PATH_VARS feature to make
CMAKE_INSTALL_LIBDIR relative to the installed location.
This change is needed for me when cross-compiling since
CMAKE_INSTALL_FULL_LIBDIR is a path inside the sysroot rather than
an absolute path to the actual includes. Without this change poppler
ends up passing a -I flag that does not exist.

This includes fallback code for CMake 2.8, which adds a bit of complexity,
since I'm not sure if raising the minimum to 3.0 (now over 8 years old)
is acceptable.

CMake: switch to GNUInstallDirs  (#1424)

* Add GNUInstallDirs for standard installation directories

Distributions are given standard variables for already existing hooks.
Multiarch libdirs is taken care of automagically.
Raises minimum cmake version by a little.

* Handle CMAKE_INSTALL_xxx being absolute paths for .pc file generation

In some cases the CMAKE_INSTAL_{BIN,MAN,DOC,LIB,INCLUDE}DIR variables
may turn out to be absolute paths in which case prepending ${prefix} in
the pkg-config .pc files will result in incorrect values.

For .pc file generation, figure out if these variables are absolute and
omit the prefix in the configured file when so.

See: https://github.com/OSGeo/PROJ/commit/ab25e4b7ed9544e668282dcd293cfaaa2e56dbdf

Merge pull request #1433 from rouault/fix_1432

opj_t1_encode_cblk(): avoid undefined behaviour on fuzzed input (fixes #1432)

opj_t1_encode_cblk(): avoid undefined behaviour on fuzzed input (fixes #1432)

Merge pull request #1431 from rouault/fix_1430

Build: fix linking of executables on some systems where TIFF/LCMS2 static libraries are not in system directories (fixes #1430)

Build: fix linking of executables on some systems where TIFF/LCMS2 static libraries are not in system directories (fixes #1430)

Note that the fix might be partial only for static-only builds (cf
comments)

Ammends PR #866 and #867

HT_DEC: Fix opj_t1_allocate_buffers malloc size error (#1426) (fixes #1413)

Merge pull request #1423 from Neumann-A/patch-1

Fix windows arm builds

Merge pull request #1421 from Biswa96/pkgconf-static

pkgconfig: Define OPJ_STATIC for static linking with pkgconf

Fix windows arm builds

pkgconfig: Define OPJ_STATIC for static linking with pkgconf

allows for the usage of $(pkgconf --static --cflags libopenjp2) to produce
the proper CFLAGS for static linking. Relies on pkgconf rather than pkg-config

HOWTO-RELEASE: update

abi_check.sh: comment OPJ_PREVIOUS_VERSION

update tools/abi-tracker/openjpeg.json

abi_check.sh: Update version number for automatic abi check

Prepare for 2.5.0 release

Separate fuzz targets to increase coverage (#1416)

Merge pull request #1386 from DavidKorczynski/cifuzz

CI: Add CIFuzz action

opj_compress.c: usage formatting fix

Merge pull request #1418 from LongerVision/master

Java Support 1.8 now...

Merge pull request #1419 from rouault/fix_ci

tools/travis-ci/install.sh: git clone with https:// to fix 'The unaut…

Rename knownfailures-Ubuntu20.04-gcc9.3.0-x86_64-Release-3rdP.txt to knownfailures-Ubuntu20.04-gcc9.4.0-x86_64-Release-3rdP.txt

.github/workflows/build.yml: force use of windows-2019 VM

tools/travis-ci/install.sh: git clone with https:// to fix 'The unauthenticated git protocol on port 9418 is no longer supported.'

Merge pull request #1410 from rouault/fix_1404

CMakeLists.txt: do not set INSTALL_NAME_DIR for MacOS builds for CMake >= 3.0 (fixes #1404)

JAVA_SOURCE_VERSION from 1.6 to 1.8

CMakeLists.txt: do not set INSTALL_NAME_DIR for MacOS builds for CMake >= 3.0 (fixes #1404)

opj_j2k_decode_tile(): avoid 'Stream too short' error in non-strict mode

Merge pull request #1408 from rouault/fix_ossfuzz_44544

Avoid integer overflows in DWT. Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=44544

Avoid integer overflows in DWT. Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=44544

Add support for partial bitstream decoding (#1407) (fixes #715)

Add a -allow-partial option to opj_decompress utility and a opj_decoder_set_strict_mode() option to the API

Co-authored-by: Chris Hafey <chafey@gmail.com>

Add .github/ISSUE_TEMPLATE.md

Merge pull request #1403 from rouault/configure_guard_bits

opj_encoder_set_extra_options(): add a GUARD_BITS=value option

opj_encoder_set_extra_options(): add a GUARD_BITS=value option

and add a -GuardBits option to opj_compress.

The recently-released SMPTE DCP Bv2.1 Application Profile (link below)
says that the number of guard bits in the QCD marker shall be 1 for 2K
content and 2 for 4K content. This change allows the number of guard bits
to be configured, so that users of openjpeg have the control they need to meet the specification.

https://ieeexplore.ieee.org/stamp/stamp.jsp?arnumber=9161348

This is an alternative implementation of https://github.com/uclouvain/openjpeg/pull/1388
that keeps ABI unchanged.

Fix potential overflow related issues spotted by LGTM code analysis (#1402)

Merge pull request #1401 from rouault/fix_1399

opj_j2k_setup_encoder(): validate number of tiles to avoid illegal values and potential overflow (fixes #1399)

opj_j2k_setup_encoder(): validate number of tiles to avoid illegal values and potential overflow (fixes #1399)

Fix unsigned vs OPJ_INT32 mismatches  (#1398)

Added check for integer overflow in get_num_images  (#1397)

As discussed in pull request 1396, added a check for integer overflow.
Change list:
Defined num_images as unsigned int
Moved the if statement to check for an empty directory to the beginning of the read directory section
Added a check to see if num images would roll back to zero when incrementing.

Avoid overflow in multiplications in utilities related to big number of files in a directory (CVE-2021-29338)  (#1396)

opj_compress/opj_uncompress: fix integer overflow in num_images (#1395)

Includes the fix for CVE-2021-29338
Credit to @kaniini based on #1346
Fixes #1338

Merge pull request #1392 from stweil/duplicate

Remove duplicate assignments in function tiftoimage

Remove duplicate assignments in function tiftoimage

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #1390 from stweil/typos

Fix some typos (found by codespell)

Format source file with typo fixes

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Fix singular/plural mismatch

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Fix some typos (found by codespell)

Signed-off-by: Stefan Weil <sw@weilnetz.de>

opj_compress: add a -TargetBitDepth switch for TIFF output (#1384)

Sometimes, given the same (16-bit TIF) input, one wants to generate a variety of J2C outputs (say, 16-, 12-, and 10-bit). This patch allows one to downsample input files, and so makes it easier to automate OpenJPEG in mass generation of J2Cs without having to pipe though an image processing program.

ci: Add CIFuzz action

Signed-off-by: David Korczynski <david@adalogics.com>

Merge pull request #1383 from rouault/deprecate_bpp

API: deprecate 'bpp' member in favor of 'prec'

API: deprecate 'bpp' member in favor of 'prec'

in opj_image_comp and opj_image_comptparm structures.

bpp was redundant with prec, and almost never set by the library, except
by opj_image_create(). This change should hopefully not impact existing,
working, users of the API, which should already have used prec to get
things working.

Fixes #1379

tests/: reformat a few files to be modified

tests/fuzzers/build_seed_corpus.sh: add htj2k files

Test decoding a .jhc file with vertically causal context variation

README.md: update badge to github action workflows [ci skip]

Test decoding a .jph test file

opj_dump.c: fix potential buffer overflow

Merge pull request #1381 from rouault/htj2k_followup

Added support for high throughput (HTJ2K) decoding.

ht_dec.c: fix likely issue on big endian hosts (untested)

opj_decompress / opj_dump: accept .jph and .jhc extensions for HTJ2K files

Added support for high throughput (HTJ2K) decoding.

There are a few limitations:
- mixed mode (HT and regular code blocks) is not supported.
- ROI in HT blocks is not supported.
- Placeholder passes are not supported.
- MultiHT sets are not support, only a singleHT set.
- there are known issues with some compliance testing files related to
  the parsing of packet header.

Merge pull request #1380 from rouault/fix_verify_indent_pr

verify-indentation.sh: fix for pull request from bar/master to foo/master

CI 'regular': fix static builds when doing perf checks

verify-indentation.sh: fix for pull request from bar/master to foo/master

Merge pull request #1376 from rouault/gha

Port continuous integration to github actions

Use standard integer types for compatibility with libtiff 4.3.0 (#1377)

appveyor.yml: disable deploy

Add github action workflows CI

Merge pull request #1375 from rouault/fix_ossfuzz_11700_30646

Avoid integer overflows in DWT.

Avoid integer overflows in DWT. Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11700 and https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=30646

Merge pull request #1373 from DimitriPapadopoulos/lgtm

LGTM warning: Comparison result is always the same

LGTM warning: Comparison result is always the same

Comparison is always true because pos >= 0.

Merge pull request #1370 from madebr/cmake/export-include-directory

cmake: add install interface include directory

Merge pull request #1371 from DimitriPapadopoulos/codespell

A couple typos found by codespell

A couple typos found by codespell

cmake: add install interface include directory

Fix segfault in src/bin/jp2/opj_decompress.c due to uninitialized pointer (fixes #1368) (#1369)

Fix code formatting

Merge pull request #1366 from sebras/fix-out-of-bounds-access

opj_j2k_is_imf_compliant: Fix out of bounds access

opj_j2k_is_imf_compliant: Fix out of bounds access.

Previously when mainlevel was parsed == 12 openjpeg would generate
a warning, but then the sublevel value would be compared to an out
of bounds element in the tabMaxSubLevelFromMainLevel array. From
this commit OpenJPEG will only use mainlevel if in range.

Merge pull request #1365 from sebras/fix-warning-formatting

opj_j2k_is_imf_compliant: Fix argument formatting for warnings.

opj_j2k_is_imf_compliant: Fix argument formatting for warnings.

Merge pull request #1361 from rouault/update_version_numbers_to_2_5

CMakeLists.txt/appveyor.yml: update version number to 2.5.0…

CMakeLists.txt/appveyor.yml: update version number to 2.5.0 to make it easier for downstream consumers to use new features

Merge pull request #1359 from rouault/TLM

Add support for enabling generation of TLM markers in encoder