libtiff: Prevent a buffer overflow in function PixarLogDecode.

libtiff: Prevent a buffer overflow in function PixarLogDecode.

Fix potential buffer write overrun in PixarLogDecode() on corrupted/unexpected 
images. The issue has been fixed in upstream (libtiff revision 1.44, 
author: erouault, commitid: 2SqWSFG5a8Ewffcz, date: 2016-06-28 23:12:19 +0800).

This CL applies the official patch to tif_pixarlog.c.

BUG=chromium:654172
R=dsinclair@chromium.org, thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/8b67b19d7e6dfb8984cc9c92ef59a81cb4edaa77

[Ke Liu, 2016-10-27 04:55:02]

Description was changed from

==========
libtiff: Prevent a buffer overflow in function PixarLogDecode.

Fix potential buffer write overrun in PixarLogDecode() on corrupted/unexpected
images. The issue has been fixed in upstream (libtiff revision 1.44, author:
erouault, commitid: 2SqWSFG5a8Ewffcz, date: 2016-06-28 23:12:19 +0800).

This CL applies the official patch to tif_pixarlog.c.

BUG=chromium:654172
R=dsinclair@chromium.org, thestig@chromium.org
==========

to

==========
libtiff: Prevent a buffer overflow in function PixarLogDecode.

Fix potential buffer write overrun in PixarLogDecode() on corrupted/unexpected 
images. The issue has been fixed in upstream (libtiff revision 1.44, 
author: erouault, commitid: 2SqWSFG5a8Ewffcz, date: 2016-06-28 23:12:19 +0800).

This CL applies the official patch to tif_pixarlog.c.

BUG=chromium:654172
R=dsinclair@chromium.org, thestig@chromium.org
==========

[Ke Liu, 2016-10-27 05:01:50]

The issue has been fixed in upstream CVS repository on Jun 28. This CL applies
the official patch to tif_pixarlog.c.

[Lei Zhang, 2016-10-27 05:26:29]

The CQ bit was checked by thestig@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-27 05:26:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Lei Zhang, 2016-10-27 05:27:38]

Let me see if the trybots like it, because the next change to tif_pixarlog.c is
to fix a signed/unsigned comparison warning.

Also, it's strange to use CVS in 2016.

[commit-bot: I haz the power, 2016-10-27 05:38:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-27 05:38:40]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-10-27 05:39:49]

lgtm

[Lei Zhang, 2016-10-27 05:40:15]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2016-10-27 05:40:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-27 05:40:40]

Description was changed from

==========
libtiff: Prevent a buffer overflow in function PixarLogDecode.

Fix potential buffer write overrun in PixarLogDecode() on corrupted/unexpected 
images. The issue has been fixed in upstream (libtiff revision 1.44, 
author: erouault, commitid: 2SqWSFG5a8Ewffcz, date: 2016-06-28 23:12:19 +0800).

This CL applies the official patch to tif_pixarlog.c.

BUG=chromium:654172
R=dsinclair@chromium.org, thestig@chromium.org
==========

to

==========
libtiff: Prevent a buffer overflow in function PixarLogDecode.

Fix potential buffer write overrun in PixarLogDecode() on corrupted/unexpected 
images. The issue has been fixed in upstream (libtiff revision 1.44, 
author: erouault, commitid: 2SqWSFG5a8Ewffcz, date: 2016-06-28 23:12:19 +0800).

This CL applies the official patch to tif_pixarlog.c.

BUG=chromium:654172
R=dsinclair@chromium.org, thestig@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/8b67b19d7e6dfb8984cc9c92ef59a81cb4ed...
==========

[commit-bot: I haz the power, 2016-10-27 05:40:41]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/8b67b19d7e6dfb8984cc9c92ef59a81cb4ed...

[Lei Zhang, 2016-10-27 05:46:09]

... and a bot config not covered by the trybots fail with a signed/unsigned
error. :\

[Lei Zhang, 2016-10-27 05:53:17]

On 2016/10/27 05:46:09, Lei Zhang wrote:
> ... and a bot config not covered by the trybots fail with a signed/unsigned
> error. :\

Follow up CL: https://codereview.chromium.org/2452293002/

[Ke Liu, 2016-10-27 06:46:45]

On 2016/10/27 05:27:38, Lei Zhang wrote:
> Let me see if the trybots like it, because the next change to tif_pixarlog.c
is
> to fix a signed/unsigned comparison warning.
> 
> Also, it's strange to use CVS in 2016.

Hello Lei, I did not notice the next fix, thanks for your follow up CL.

I think it's weird too. It's my first time using the CVS :)