Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(109)

Issue 2369433002: Bail out on negative a0 values in FaxG4GetRow(). (Closed)

Created:
4 years, 3 months ago by Lei Zhang
Modified:
4 years, 2 months ago
Reviewers:
kcwu
CC:
pdfium-reviews_googlegroups.com
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Bail out on negative a0 values in FaxG4GetRow(). BUG=648935

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -10 lines) Patch
M core/fxcodec/codec/fx_codec_fax.cpp View 3 chunks +7 lines, -10 lines 0 comments Download

Messages

Total messages: 8 (6 generated)
kcwu
I feel it should reject negative |columns| somewhere.
4 years, 3 months ago (2016-09-23 08:22:42 UTC) #6
kcwu
4 years, 3 months ago (2016-09-23 08:25:30 UTC) #7
On 2016/09/23 08:22:42, kcwu wrote:
> I feel it should reject negative |columns| somewhere.

p.s. negative |columns| is the source of other negative values.

Powered by Google App Engine
This is Rietveld 408576698