diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-07-15 14:01:46 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2015-08-03 19:50:38 +0200 |
commit | 556514fd47f48aba7f81fc2dc4b45315b745a628 (patch) | |
tree | 19c3902fbf553eeb0efd7ef756eb7e28f8dea13f | |
parent | 54a1bd325673b0577e91d9e506add023a646b363 (diff) |
avoid hang in short pbm
Change-Id: I9b7f0832a4dc231e1e8f963858c155e3cd392667
(cherry picked from commit b8637e67d6d39e47d22cfce496000288f0dc58d8)
Reviewed-on: https://gerrit.libreoffice.org/17085
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
-rw-r--r-- | filter/qa/cppunit/data/pbm/fail/.gitignore | 0 | ||||
-rw-r--r-- | filter/qa/cppunit/data/pbm/fail/hang-1.pbm | bin | 0 -> 266 bytes | |||
-rw-r--r-- | filter/qa/cppunit/data/pbm/indeterminate/.gitignore | 1 | ||||
-rw-r--r-- | filter/qa/cppunit/data/pbm/pass/.gitignore | 0 | ||||
-rw-r--r-- | filter/qa/cppunit/data/pbm/pass/rhbz160429-1.pbm | bin | 0 -> 456 bytes | |||
-rw-r--r-- | filter/qa/cppunit/filters-ppm-test.cxx | 4 | ||||
-rw-r--r-- | filter/source/graphicfilter/ipbm/ipbm.cxx | 2 |
7 files changed, 6 insertions, 1 deletions
diff --git a/filter/qa/cppunit/data/pbm/fail/.gitignore b/filter/qa/cppunit/data/pbm/fail/.gitignore new file mode 100644 index 000000000000..e69de29bb2d1 --- /dev/null +++ b/filter/qa/cppunit/data/pbm/fail/.gitignore diff --git a/filter/qa/cppunit/data/pbm/fail/hang-1.pbm b/filter/qa/cppunit/data/pbm/fail/hang-1.pbm Binary files differnew file mode 100644 index 000000000000..21742d204c79 --- /dev/null +++ b/filter/qa/cppunit/data/pbm/fail/hang-1.pbm diff --git a/filter/qa/cppunit/data/pbm/indeterminate/.gitignore b/filter/qa/cppunit/data/pbm/indeterminate/.gitignore new file mode 100644 index 000000000000..e9c5b1765b8f --- /dev/null +++ b/filter/qa/cppunit/data/pbm/indeterminate/.gitignore @@ -0,0 +1 @@ +*.ppm-* diff --git a/filter/qa/cppunit/data/pbm/pass/.gitignore b/filter/qa/cppunit/data/pbm/pass/.gitignore new file mode 100644 index 000000000000..e69de29bb2d1 --- /dev/null +++ b/filter/qa/cppunit/data/pbm/pass/.gitignore diff --git a/filter/qa/cppunit/data/pbm/pass/rhbz160429-1.pbm b/filter/qa/cppunit/data/pbm/pass/rhbz160429-1.pbm Binary files differnew file mode 100644 index 000000000000..d6e3fc6349b8 --- /dev/null +++ b/filter/qa/cppunit/data/pbm/pass/rhbz160429-1.pbm diff --git a/filter/qa/cppunit/filters-ppm-test.cxx b/filter/qa/cppunit/filters-ppm-test.cxx index 9b281abff231..e8bef145ada6 100644 --- a/filter/qa/cppunit/filters-ppm-test.cxx +++ b/filter/qa/cppunit/filters-ppm-test.cxx @@ -62,6 +62,10 @@ void PpmFilterTest::testCVEs() testDir(OUString(), getURLFromSrc("/filter/qa/cppunit/data/ppm/"), OUString()); + + testDir(OUString(), + getURLFromSrc("/filter/qa/cppunit/data/pbm/"), + OUString()); } CPPUNIT_TEST_SUITE_REGISTRATION(PpmFilterTest); diff --git a/filter/source/graphicfilter/ipbm/ipbm.cxx b/filter/source/graphicfilter/ipbm/ipbm.cxx index 63a8b094633c..720bc9f527fb 100644 --- a/filter/source/graphicfilter/ipbm/ipbm.cxx +++ b/filter/source/graphicfilter/ipbm/ipbm.cxx @@ -195,7 +195,7 @@ bool PBMReader::ImplReadHeader() } while ( bFinished == false ) { - if ( mrPBM.GetError() ) + if (!mrPBM.good()) return false; mrPBM.ReadUChar( nDat ); |