Bug 51137 - IOSurface bounds checks on bytesPerRow and allocSize are incorrect
Summary: IOSurface bounds checks on bytesPerRow and allocSize are incorrect
Alias: None
Product: WebKit
Classification: Unclassified
Component: Canvas (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P2 Normal
Assignee: Matthew Delaney
Depends on:
Reported: 2010-12-15 14:56 PST by Matthew Delaney
Modified: 2010-12-15 15:28 PST (History)
2 users (show)

See Also:

Patch (1.44 KB, patch)
2010-12-15 15:05 PST, Matthew Delaney
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Delaney 2010-12-15 14:56:34 PST
Since the IOSurfaceAlignProperty functions may align bytesPerRow and allocSize for reasons other than not allowing the values to be too large, bounds checking off of their returned values is not correct.

This patch will remove those incorrect bounds checks.
Comment 1 Matthew Delaney 2010-12-15 15:05:21 PST
Created attachment 76690 [details]
Comment 2 Eric Seidel (no email) 2010-12-15 15:15:41 PST
Comment on attachment 76690 [details]

Which tests?  On which platform?
Comment 3 Simon Fraser (smfr) 2010-12-15 15:20:18 PST
Comment on attachment 76690 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=76690&action=review

> WebCore/ChangeLog:8
> +        No new tests added - many current canvas layout tests fail because of this incorrect bounds check.

You should clarify that tests will only fail for those using this new functionality.
Comment 4 Matthew Delaney 2010-12-15 15:28:20 PST
Yea, exactly. Only on my local build using this new functionality (IOSurfaces) are any of the usual tests failing. This patch will fix those - not any tests failing on the bots. This is now reflected in the changelog.
Comment 5 Matthew Delaney 2010-12-15 15:28:31 PST
Committed r74148: <http://trac.webkit.org/changeset/74148>