diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2020-08-20 21:54:25 +0200 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-11-24 21:29:01 +0100 |
| commit | 5f159d4f8524cd466e124ca8366d71566cb883c4 (patch) | |
| tree | 169ed92e123837e787ca6b081932706ffc023ea5 | |
| parent | 43433f42017014a472a253314a6ac58a6774dced (diff) | |
fix unit-test with non-pdfium build
otherwise testSwappingPageNumber will crash accessing out of
bounds element in vector.
Change-Id: I07499e79d523931e1f3cc02a5150f033e1ea9578
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101094
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 17798a503bc67f2a38f4b558e37447d1ea8f4623)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106511
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
| -rw-r--r-- | filter/source/pdf/pdfdecomposer.cxx | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/filter/source/pdf/pdfdecomposer.cxx b/filter/source/pdf/pdfdecomposer.cxx index e053fd215369..e54206e4c175 100644 --- a/filter/source/pdf/pdfdecomposer.cxx +++ b/filter/source/pdf/pdfdecomposer.cxx @@ -70,7 +70,10 @@ uno::Sequence<uno::Reference<graphic::XPrimitive2D>> SAL_CALL XPdfDecomposer::ge nPageIndex = 0; std::vector<Bitmap> aBitmaps; - vcl::RenderPDFBitmaps(xPdfData.getConstArray(), xPdfData.getLength(), aBitmaps, nPageIndex, 1); + int rv = vcl::RenderPDFBitmaps(xPdfData.getConstArray(), xPdfData.getLength(), aBitmaps, + nPageIndex, 1); + if (rv == 0) + return {}; // happens if we do not have PDFium BitmapEx aReplacement(aBitmaps[0]); |