summaryrefslogtreecommitdiff
path: root/fpdfsdk/fpdf_transformpage.cpp
diff options
context:
space:
mode:
authorShirleen Lou <xlou@chromium.org>2017-11-18 00:13:45 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-11-18 00:13:45 +0000
commit07b4727a34d2f4aff851f0dc420faf617caca220 (patch)
treeb0290d7c6584be36e76a6892bd285afaf1b99461 /fpdfsdk/fpdf_transformpage.cpp
parentec2209da158750beeeaeaab5dd68e58424921c4a (diff)
downloadpdfium-07b4727a34d2f4aff851f0dc420faf617caca220.tar.xz
Make clip optional in FPDFPage_TransFormWithClip.chromium/3273
Add checks for matrix and clipRect in function FPDFPage_TransFormWithClip. If both are null, abort transformation. If either is null, skip that particular transaction. Change-Id: I65611508c064c0b929a11eace99ad532d3402138 Reviewed-on: https://pdfium-review.googlesource.com/18710 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fpdfsdk/fpdf_transformpage.cpp')
-rw-r--r--fpdfsdk/fpdf_transformpage.cpp20
1 files changed, 14 insertions, 6 deletions
diff --git a/fpdfsdk/fpdf_transformpage.cpp b/fpdfsdk/fpdf_transformpage.cpp
index b607dffc4e..4f29fd69d6 100644
--- a/fpdfsdk/fpdf_transformpage.cpp
+++ b/fpdfsdk/fpdf_transformpage.cpp
@@ -106,19 +106,27 @@ FPDF_EXPORT FPDF_BOOL FPDF_CALLCONV
FPDFPage_TransFormWithClip(FPDF_PAGE page,
FS_MATRIX* matrix,
FS_RECTF* clipRect) {
+ if (!matrix && !clipRect)
+ return false;
+
CPDF_Page* pPage = CPDFPageFromFPDFPage(page);
if (!pPage)
return false;
std::ostringstream textBuf;
textBuf << "q ";
- CFX_FloatRect rect = CFXFloatRectFromFSRECTF(*clipRect);
- rect.Normalize();
- textBuf << ByteString::Format("%f %f %f %f re W* n ", rect.left, rect.bottom,
- rect.Width(), rect.Height());
- textBuf << ByteString::Format("%f %f %f %f %f %f cm ", matrix->a, matrix->b,
- matrix->c, matrix->d, matrix->e, matrix->f);
+ if (clipRect) {
+ CFX_FloatRect rect = CFXFloatRectFromFSRECTF(*clipRect);
+ rect.Normalize();
+
+ textBuf << ByteString::Format("%f %f %f %f re W* n ", rect.left,
+ rect.bottom, rect.Width(), rect.Height());
+ }
+ if (matrix) {
+ textBuf << ByteString::Format("%f %f %f %f %f %f cm ", matrix->a, matrix->b,
+ matrix->c, matrix->d, matrix->e, matrix->f);
+ }
CPDF_Dictionary* pPageDict = pPage->m_pFormDict.Get();
CPDF_Object* pContentObj = GetPageContent(pPageDict);