diff options
author | Nicolas Pena <npm@chromium.org> | 2017-08-14 10:36:01 -0400 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-08-14 15:02:17 +0000 |
commit | 0bd847232a1f430c70dd9d8df177ce68a3cde010 (patch) | |
tree | 15cec8c11493549f1974ae2f6aeac58234c176d2 /third_party/lcms/0016-check-LUT-and-MPE.patch | |
parent | dff02cee2d2410d81a55c59345fb38b5aac8a457 (diff) | |
download | pdfium-0bd847232a1f430c70dd9d8df177ce68a3cde010.tar.xz |
LCMS: upgrade to 2.8
This CL upgrades LCMS from version 2.6 to 2.8. All changes from LCMS
original version 2.8 are stored in patch files:
- Patch 0: memory management modifications to use PDFium methods. This
was previously not in any patch, so the changes were manually applied.
- Patches 1-5: new patch files corresponding to old changes that can be
seen in the history, but did not previously have patch files.
- Patches 6-25: previous patches (patch numbers shifted by 6). The one
for from16-to-8-overflow.patch was deleted as it was already upstream.
Some patches did not apply cleanly so their .patch files were modified.
- Patch 26: as I just moved files directly, unsupported characters were
moved in unchanged, so I had to fix all of them: e with tilde and
other characters were replaced to allow compilation on Windows.
- Patch 27: Went over the code and re-applied changes that included
comments clearly indicating this was Foxit. These changes are all
already seen in the initial PDFium commit.
Change-Id: Ic1d84e54803ef9e6b280ef7619bbf0b757312fbf
Reviewed-on: https://pdfium-review.googlesource.com/10590
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'third_party/lcms/0016-check-LUT-and-MPE.patch')
-rw-r--r-- | third_party/lcms/0016-check-LUT-and-MPE.patch | 170 |
1 files changed, 0 insertions, 170 deletions
diff --git a/third_party/lcms/0016-check-LUT-and-MPE.patch b/third_party/lcms/0016-check-LUT-and-MPE.patch deleted file mode 100644 index e2efe5a3a7..0000000000 --- a/third_party/lcms/0016-check-LUT-and-MPE.patch +++ /dev/null @@ -1,170 +0,0 @@ -diff --git a/third_party/lcms/src/cmslut.c b/third_party/lcms/src/cmslut.c -index 9b0eb4b54..19d43361f 100644 ---- a/third_party/lcms/src/cmslut.c -+++ b/third_party/lcms/src/cmslut.c -@@ -1255,21 +1255,39 @@ cmsStage* CMSEXPORT cmsStageDup(cmsStage* mpe) - // *********************************************************************************************************** - - // This function sets up the channel count -- - static --void BlessLUT(cmsPipeline* lut) -+cmsBool BlessLUT(cmsPipeline* lut) - { - // We can set the input/ouput channels only if we have elements. - if (lut ->Elements != NULL) { - -- cmsStage *First, *Last; -+ cmsStage* prev; -+ cmsStage* next; -+ cmsStage* First; -+ cmsStage* Last; - - First = cmsPipelineGetPtrToFirstStage(lut); - Last = cmsPipelineGetPtrToLastStage(lut); - -- if (First != NULL)lut ->InputChannels = First ->InputChannels; -- if (Last != NULL) lut ->OutputChannels = Last ->OutputChannels; -+ if (First == NULL || Last == NULL) return FALSE; -+ -+ lut->InputChannels = First->InputChannels; -+ lut->OutputChannels = Last->OutputChannels; -+ -+ // Check chain consistency -+ prev = First; -+ next = prev->Next; -+ -+ while (next != NULL) -+ { -+ if (next->InputChannels != prev->OutputChannels) -+ return FALSE; -+ -+ next = next->Next; -+ prev = prev->Next; -+ } - } -+ return TRUE; - } - - -@@ -1331,6 +1349,7 @@ cmsPipeline* CMSEXPORT cmsPipelineAlloc(cmsContext ContextID, cmsUInt32Number In - { - cmsPipeline* NewLUT; - -+ // A value of zero in channels is allowed as placeholder - if (InputChannels >= cmsMAXCHANNELS || - OutputChannels >= cmsMAXCHANNELS) return NULL; - -@@ -1348,7 +1367,11 @@ cmsPipeline* CMSEXPORT cmsPipelineAlloc(cmsContext ContextID, cmsUInt32Number In - NewLUT ->Data = NewLUT; - NewLUT ->ContextID = ContextID; - -- BlessLUT(NewLUT); -+ if (!BlessLUT(NewLUT)) -+ { -+ _cmsFree(ContextID, NewLUT); -+ return NULL; -+ } - - return NewLUT; - } -@@ -1454,7 +1477,12 @@ cmsPipeline* CMSEXPORT cmsPipelineDup(const cmsPipeline* lut) - - NewLUT ->SaveAs8Bits = lut ->SaveAs8Bits; - -- BlessLUT(NewLUT); -+ if (!BlessLUT(NewLUT)) -+ { -+ _cmsFree(lut->ContextID, NewLUT); -+ return NULL; -+ } -+ - return NewLUT; - } - -@@ -1491,8 +1519,7 @@ int CMSEXPORT cmsPipelineInsertStage(cmsPipeline* lut, cmsStageLoc loc, cmsStage - return FALSE; - } - -- BlessLUT(lut); -- return TRUE; -+ return BlessLUT(lut); - } - - // Unlink an element and return the pointer to it -@@ -1547,6 +1574,7 @@ void CMSEXPORT cmsPipelineUnlinkStage(cmsPipeline* lut, cmsStageLoc loc, cmsStag - else - cmsStageFree(Unlinked); - -+ // May fail, but we ignore it - BlessLUT(lut); - } - -@@ -1573,8 +1601,7 @@ cmsBool CMSEXPORT cmsPipelineCat(cmsPipeline* l1, const cmsPipeline* l2) - return FALSE; - } - -- BlessLUT(l1); -- return TRUE; -+ return BlessLUT(l1); - } - - -diff --git a/third_party/lcms/src/cmstypes.c b/third_party/lcms/src/cmstypes.c -index e5ed06c33..0256e247b 100644 ---- a/third_party/lcms/src/cmstypes.c -+++ b/third_party/lcms/src/cmstypes.c -@@ -1755,8 +1755,8 @@ void *Type_LUT8_Read(struct _cms_typehandler_struct* self, cmsIOHANDLER* io, cms - if (!_cmsReadUInt8Number(io, NULL)) goto Error; - - // Do some checking -- if (InputChannels > cmsMAXCHANNELS) goto Error; -- if (OutputChannels > cmsMAXCHANNELS) goto Error; -+ if (InputChannels == 0 || InputChannels > cmsMAXCHANNELS) goto Error; -+ if (OutputChannels == 0 || OutputChannels > cmsMAXCHANNELS) goto Error; - - // Allocates an empty Pipeline - NewLUT = cmsPipelineAlloc(self ->ContextID, InputChannels, OutputChannels); -@@ -2048,8 +2048,8 @@ void *Type_LUT16_Read(struct _cms_typehandler_struct* self, cmsIOHANDLER* io, cm - if (!_cmsReadUInt8Number(io, NULL)) return NULL; - - // Do some checking -- if (InputChannels > cmsMAXCHANNELS) goto Error; -- if (OutputChannels > cmsMAXCHANNELS) goto Error; -+ if (InputChannels == 0 || InputChannels > cmsMAXCHANNELS) goto Error; -+ if (OutputChannels == 0 || OutputChannels > cmsMAXCHANNELS) goto Error; - - // Allocates an empty LUT - NewLUT = cmsPipelineAlloc(self ->ContextID, InputChannels, OutputChannels); -@@ -2486,7 +2486,10 @@ void* Type_LUTA2B_Read(struct _cms_typehandler_struct* self, cmsIOHANDLER* io, c - if (!_cmsReadUInt32Number(io, &offsetC)) return NULL; - if (!_cmsReadUInt32Number(io, &offsetA)) return NULL; - -- // Allocates an empty LUT -+ if (inputChan == 0 || inputChan >= cmsMAXCHANNELS) return NULL; -+ if (outputChan == 0 || outputChan >= cmsMAXCHANNELS) return NULL; -+ -+ // Allocates an empty LUT - NewLUT = cmsPipelineAlloc(self ->ContextID, inputChan, outputChan); - if (NewLUT == NULL) return NULL; - -@@ -2794,6 +2797,9 @@ void* Type_LUTB2A_Read(struct _cms_typehandler_struct* self, cmsIOHANDLER* io, c - if (!_cmsReadUInt8Number(io, &inputChan)) return NULL; - if (!_cmsReadUInt8Number(io, &outputChan)) return NULL; - -+ if (inputChan == 0 || inputChan >= cmsMAXCHANNELS) return NULL; -+ if (outputChan == 0 || outputChan >= cmsMAXCHANNELS) return NULL; -+ - // Padding - if (!_cmsReadUInt16Number(io, NULL)) return NULL; - -@@ -4443,6 +4449,9 @@ void *Type_MPE_Read(struct _cms_typehandler_struct* self, cmsIOHANDLER* io, cmsU - if (!_cmsReadUInt16Number(io, &InputChans)) return NULL; - if (!_cmsReadUInt16Number(io, &OutputChans)) return NULL; - -+ if (InputChans == 0 || InputChans >= cmsMAXCHANNELS) return NULL; -+ if (OutputChans == 0 || OutputChans >= cmsMAXCHANNELS) return NULL; -+ - // Allocates an empty LUT - NewLUT = cmsPipelineAlloc(self ->ContextID, InputChans, OutputChans); - if (NewLUT == NULL) return NULL; |