diff options
| author | Matthieu Darbois <mayeut@users.noreply.github.com> | 2016-09-08 00:24:15 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-09-08 00:24:15 +0200 |
| commit | e078172b1c3f98d2219c37076b238fb759c751ea (patch) | |
| tree | ac6d5c8489af7fc85b8c9b5d8854ab5c69d2cc47 /tests | |
| parent | acfb307df43a811cc56c035d0dc651dac8c9e7de (diff) | |
Add sanity check for tile coordinates (#823)
Coordinates are casted from OPJ_UINT32 to OPJ_INT32
Add sanity check for negative values and upper bound becoming lower
than lower bound.
See also
https://pdfium.googlesource.com/pdfium/+/b6befb2ed2485a3805cddea86dc7574510178ea9
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/compare_dump_files.c | 14 | ||||
| -rw-r--r-- | tests/nonregression/test_suite.ctest.in | 3 |
2 files changed, 10 insertions, 7 deletions
diff --git a/tests/compare_dump_files.c b/tests/compare_dump_files.c index 946c92a5..7d222708 100644 --- a/tests/compare_dump_files.c +++ b/tests/compare_dump_files.c @@ -118,10 +118,10 @@ int main(int argc, char **argv) test_cmp_parameters inParam; FILE *fbase=NULL, *ftest=NULL; int same = 0; - char lbase[256]; - char strbase[256]; - char ltest[256]; - char strtest[256]; + char lbase[512]; + char strbase[512]; + char ltest[512]; + char strtest[512]; if( parse_cmdline_cmp(argc, argv, &inParam) == 1 ) { @@ -154,9 +154,9 @@ int main(int argc, char **argv) while (fgets(lbase, sizeof(lbase), fbase) && fgets(ltest,sizeof(ltest),ftest)) { - int nbase = sscanf(lbase, "%255[^\r\n]", strbase); - int ntest = sscanf(ltest, "%255[^\r\n]", strtest); - assert( nbase != 255 && ntest != 255 ); + int nbase = sscanf(lbase, "%511[^\r\n]", strbase); + int ntest = sscanf(ltest, "%511[^\r\n]", strtest); + assert( nbase != 511 && ntest != 511 ); if( nbase != 1 || ntest != 1 ) { fprintf(stderr, "could not parse line from files\n" ); diff --git a/tests/nonregression/test_suite.ctest.in b/tests/nonregression/test_suite.ctest.in index bbf1e77f..2dfabfbc 100644 --- a/tests/nonregression/test_suite.ctest.in +++ b/tests/nonregression/test_suite.ctest.in @@ -566,3 +566,6 @@ opj_decompress -i @INPUT_NR_PATH@/issue726.j2k -o @TEMP_PATH@/issue726.png !opj_decompress -i @INPUT_NR_PATH@/issue775-2.j2k -o @TEMP_PATH@/issue775-2.png # issue 818 opj_decompress -i @INPUT_NR_PATH@/issue818.jp2 -o @TEMP_PATH@/issue818.png +# issue 823 (yes, not a typo, test image is issue822) +!opj_decompress -i @INPUT_NR_PATH@/issue822.jp2 -o @TEMP_PATH@/issue822.png + |