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

Issue 6576032: Check for excessive dimensions. (Closed)

Created:
9 years, 10 months ago by Chris Evans
Modified:
9 years, 6 months ago
Reviewers:
jschuh, cevans
CC:
chromium-reviews, apatrick_chromium
Visibility:
Public.

Description

Check for excessive dimensions. BUG=72173 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76028

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Chris Evans
As discussed!
9 years, 10 months ago (2011-02-24 22:33:30 UTC) #1
jschuh
SGTM. One dumb question though. Are zero dimensions allowed? If not, you could simplify to ...
9 years, 10 months ago (2011-02-25 02:06:40 UTC) #2
cevans
9 years, 10 months ago (2011-02-25 02:34:02 UTC) #3
On Thu, Feb 24, 2011 at 6:06 PM, <jschuh@chromium.org> wrote:

> SGTM. One dumb question though. Are zero dimensions allowed?


I do not know; and I'm playing it conservatively after our previous
experiments here, for now :)


> If not, you could
> simplify to the following (just for readability):
>
> if (w <= 0 || h <= 0 || h >= (INT_MAX / 4) / w)
>
>
>
> http://codereview.chromium.org/6576032/
>

Powered by Google App Engine
This is Rietveld 408576698