Fix bmp bug exposed by fuzzer

Fix bmp bug exposed by fuzzer

BUG=skia:5820
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391933004

Committed: https://skia.googlesource.com/skia/+/d567a815b0eef3f0cdaee6b45bac71414a437f0e

[msarett, 2016-10-04 23:42:47]

Description was changed from

==========
Fix bmp bug exposed by fuzzer

BUG=skia:5820
==========

to

==========
Fix bmp bug exposed by fuzzer

BUG=skia:5820
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391933004
==========

[msarett, 2016-10-04 23:43:40]

msarett@google.com changed reviewers:
+ kjlubick@google.com, scroggo@google.com

[msarett, 2016-10-04 23:43:40]

The bug and the fix are obvious.  I'm working on finding a test image though.

[scroggo, 2016-10-05 13:51:10]

LGTM

On 2016/10/04 23:43:40, msarett wrote:
> The bug and the fix are obvious.  I'm working on finding a test image though.

Can't you use the image the fuzzer created?

[msarett, 2016-10-05 13:59:27]

On 2016/10/05 13:51:10, scroggo wrote:
> LGTM
> 
> On 2016/10/04 23:43:40, msarett wrote:
> > The bug and the fix are obvious.  I'm working on finding a test image
though.
> 
> Can't you use the image the fuzzer created?

Yes, I guess the fuzzer is the test here.  I'm mostly just annoyed that I can't
find an image that exercises this code.

[msarett, 2016-10-05 13:59:31]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-05 13:59:33]

Dry run: 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-05 14:16:42]

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

[commit-bot: I haz the power, 2016-10-05 14:16:43]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-10-05 14:18:23]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-10-05 14:18:32]

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-05 14:19:29]

Description was changed from

==========
Fix bmp bug exposed by fuzzer

BUG=skia:5820
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391933004
==========

to

==========
Fix bmp bug exposed by fuzzer

BUG=skia:5820
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391933004

Committed:
https://skia.googlesource.com/skia/+/d567a815b0eef3f0cdaee6b45bac71414a437f0e
==========

[commit-bot: I haz the power, 2016-10-05 14:19:30]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/d567a815b0eef3f0cdaee6b45bac71414a437f0e