Commit bacac8f7 authored by Xianda Sun's avatar Xianda Sun Committed by Commit Bot

Fix hit test query fuzzer

This patch fixes a bug in previous fuzzer patch. We need to use "and"
operand to make sure the flag contains kHitTestAsk.

Bug: 905463
Change-Id: I13935185756dfef4474b94975bfec833469bdac1
Reviewed-on: https://chromium-review.googlesource.com/c/1351271Reviewed-by: default avatarRia Jiang <riajiang@chromium.org>
Commit-Queue: Xianda Sun <sunxd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610871}
parent 798c47c8
...@@ -30,7 +30,7 @@ void AddHitTestRegion(base::FuzzedDataProvider* fuzz, ...@@ -30,7 +30,7 @@ void AddHitTestRegion(base::FuzzedDataProvider* fuzz,
uint32_t flags = GetNextUInt32(fuzz); uint32_t flags = GetNextUInt32(fuzz);
// The reasons' value is kNotAsyncHitTest if the flag's value is kHitTestAsk. // The reasons' value is kNotAsyncHitTest if the flag's value is kHitTestAsk.
uint32_t reasons = uint32_t reasons =
(flags | viz::HitTestRegionFlags::kHitTestAsk) (flags & viz::HitTestRegionFlags::kHitTestAsk)
? fuzz->ConsumeUint32InRange(1, std::numeric_limits<uint32_t>::max()) ? fuzz->ConsumeUint32InRange(1, std::numeric_limits<uint32_t>::max())
: viz::AsyncHitTestReasons::kNotAsyncHitTest; : viz::AsyncHitTestReasons::kNotAsyncHitTest;
gfx::Rect rect(fuzz->ConsumeUint8(), fuzz->ConsumeUint8(), gfx::Rect rect(fuzz->ConsumeUint8(), fuzz->ConsumeUint8(),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment