Merge pull request #1518 from dg0yt/static-windows

Fix export of OPJ_STATIC, cleanup

Merge pull request #1519 from stweil/crlf

Fix line endings (CRLF -> LF)

Fix line endings (CRLF -> LF)

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

Refactor build of both types of linkage

Fix export of OPJ_STATIC macro

Merge pull request #1517 from stweil/typos

Fix some typos (found by `codespell` and `typos`)

Merge pull request #1516 from stweil/actions

Update GitHub checkout action to latest version

Fix some typos (found by `codespell` and `typos`)

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

Update GitHub checkout action to latest version

This fixes lots of warnings like this one:

    The following actions uses node12 which is deprecated and will be forced to run on node16:
    actions/checkout@v2. For more info:
    https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

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

Merge pull request #1515 from rouault/fix_1514

openjpeg.h: make sure to include opj_config.h (fixes #1514)

opj_config.h.cmake.in: add inclusion guard

openjpeg.h: make sure to include opj_config.h (fixes #1514)

It was previously included by openjpeg.h

update HOWTO-RELEASE

update HOWTO-RELEASE

abi_check.sh: comment OPJ_PREVIOUS_VERSION

abi_check.sh: Update version number for automatic abi check (2.5.0->2.5.1)

Prepare for OpenJPEG v2.5.1 release

Merge pull request #1511 from rouault/fix_1488_bis

opj_t2_read_packet_header(): avoid unsigned integer overflow

opj_t2_read_packet_header(): avoid unsigned integer overflow (alternate fix to https://github.com/uclouvain/openjpeg/pull/1488)

Merge pull request #1510 from rouault/fix_1505

opj_dwt_decode_tile(): avoid potential UndefinedBehaviorSanitizer 'applying zero offset to null pointer' (fixes #1505)

Merge pull request #1507 from kmilos/patch-1

Always install pkgconfig files

Merge pull request #1509 from rouault/fix_sycc420_to_rgb

opj_decompress: fix off-by-one read heap-buffer-overflow in sycc420_to_rgb() when x0 and y0 are odd (CVE-2021-3575, fixes #1347)

opj_dwt_decode_tile(): avoid potential UndefinedBehaviorSanitizer 'applying zero offset to null pointer' (fixes #1505)

opj_decompress: fix off-by-one read heap-buffer-overflow in sycc420_to_rgb() when x0 and y0 are odd (CVE-2021-3575, fixes #1347)

Remove BUILD_PKGCONFIG_FILES

Always install pkgconfig files

They are supported on all platforms

Merge pull request #1493 from domin144/cmake_version_file

Add cmake version file

Add cmake version file

Merge pull request #1503 from domin144/drop_cmake_less_3.5

CMake: drop support for cmake < 3.5

CMake: drop support for cmake < 3.5

Compatibility for cmake version < 3.5 is now deprecated in cmake and
specifing version below 3.5 in cmake_minimum_required produces a
warning.[1]

[1] https://cmake.org/cmake/help/latest/release/3.27.html#deprecated-and-removed-features

Merge pull request #1502 from scaramallion/fix-compiler-error

Fix compiler error on Windows

Fix compiler error on Windows

ht_dec.c: Improve MSVC arm64 popcount performance (#1479)

Use NEON instructions for ARM64 (implementation based on microsoft/STL#2127).

Godbolt output here: https://godbolt.org/z/q7GPTqT14

Merge pull request #1496 from rouault/fix_1480

opj_tcd_dc_level_shift_decode(): avoid increment nullptr (fixes #1480)

Merge pull request #1490 from headshog/fix-j2k

Integer Overflow at j2k.c:3962

Merge pull request #1491 from headshog/fix-j2k-header

Integer Overflow at j2k.c:11114

Merge pull request #1492 from headshog/fix-ht-dec

fix ht_dec.c:1215

opj_tcd_dc_level_shift_decode(): avoid increment nullptr (fixes #1480)

(likely harmless issue as we don't dereference it)

Merge pull request #1484 from mayeut/mandatory-inttypes-stdint

Require `stdint.h` & `inttypes.h`

Merge pull request #1494 from tomoaki0705/suppress_warning_clang

suppress warning during build using clang

opj_t1_ht_decode_cblk(): avoid memcpy() with 0-input size and nullptr destination. fix ht_dec.c:1215 (#1492)

fix j2k.c:11114

opj_j2k_merge_ppm(): avoid unsigned-integer-overflow at j2k.c:3962 (#1490)

Merge pull request #1495 from rouault/update_kdu

Fix CI

tools/travis-ci/install.sh: workaround issue with jpylyzer on Mac

suppress warning during build using clang

CI: update KDU to new version (older one no longer available)

Fixed typos (#1486)

Require `stdint.h` & `inttypes.h`

With #1450 which goes with 480cc9d49775d018c19ce0e01f6be27858d63d39 "Remove support for non-C99 compilers (like VS2010) that don't support snprintf()",
support for MSVC versions prior to vs2015 is dropped: https://stackoverflow.com/questions/2915672/snprintf-and-visual-studio-2010

This means that all supported MSVC versions do have `stdint.h` & `inttypes.h` now.
For non windows platforms, those headers were already mandatory.

Make them mandatory for all builds.

Merge pull request #1482 from mayeut/use-opj_uint_ceildiv

fix: use `opj_uint_ceildiv` instead of `opj_int_ceildiv` when necessary

chore: GHA Ubuntu 22.04 now uses gcc 11.4.0

fix: use `opj_uint_ceildiv` instead of `opj_int_ceildiv` when necessary

There are a bunch of loc where we can see a usage of `opj_int_ceildiv`:
```
(OPJ_UINT32)opj_int_ceildiv((OPJ_INT32)a, (OPJ_INT32)b);
```
where a & b are `OPJ_UINT32`.

This can lead to overflow/underflow for some a/b combinations.
Replace those calls by `opj_uint_ceildiv` instead to always get a correct result.

This also allows some valid single tile images with huge tile size to be decoded properly.
Fix #1438

Merge pull request #1463 from rouault/fix_570

opj_jp2_read_header(): move setting color_space here instead in opj_jp2_decode()/get_tile() (fixes #570)

opj_jp2_read_header(): move setting icc_profile here instead in opj_jp2_decode()/get_tile() + add unit test (fixes #570)

opj_jp2_read_header(): move setting color_space here instead in opj_jp2_decode()/get_tile() (fixes #570)

Merge pull request #1462 from sebras/master

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

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

And fix strict-prototypes/missing-prototypes warnings.

Merge pull request #1450 from markmentovai/no_sprintf

openjp2/j2k: replace sprintf calls with snprintf

openjp2/j2k: replace sprintf calls with snprintf

This makes it possible to build j2k.c without warnings using the macOS
13 SDK. Calls to sprintf are replaced with snprintf, passing appropriate
buffer sizes.

It doesn’t appear that any of the changed uses of sprintf were actually
unsafe, so no behavior change is expected aside from SDK compatibility.

The macOS 13 SDK deprecates sprintf as it’s difficult to use safely. The
deprecation warning message is visible when building C++, but it is not
normally visible when building plain C code due to a quirk in how
sprintf is declared in the SDK. However, the deprecation message is
visible when building plain C under Address Sanitizer
(-fsanitize=address). This discrepancy was discovered at
https://crbug.com/1381706 and reported to Apple with a copy at
https://openradar.appspot.com/FB11761475.

The macOS 13 SDK is packaged in Xcode 14.1, released on 2022-11-01. This
also affects the iOS 16 SDK and other 2022-era Apple OS SDKs packaged in
Xcode 14.0, released on 2022-09-12.

j2k.c is visible to the Chromium build via PDFium, and this change is
needed to allow Chromium to move forward to the macOS 13 SDK.

This change is limited to src/lib/openjp2. Other uses of sprintf were
found throughout openjpeg.

Remove support for non-C99 compilers (like VS2010) that don't support snprintf()

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