Closed
Bug 1404174
Opened 7 years ago
Closed 7 years ago
fsanitize=enum (ubsan) runtime errors for nsEventStatus
Categories
(Core :: Panning and Zooming, defect, P3)
Core
Panning and Zooming
Tracking
()
RESOLVED
FIXED
mozilla58
People
(Reporter: arthur, Assigned: arthur)
References
(Blocks 1 open bug)
Details
(Whiteboard: [tor][gfx-noted])
Attachments
(1 file)
|
884 bytes,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
An -fsanitize=enum run gives errors like: [task 2017-09-25T00:24:35.119Z] 00:24:35 INFO - GECKO(1097) | /builds/worker/workspace/build/src/gfx/layers/apz/util/InputAPZContext.cpp:58:18: runtime error: load of value 2788801888, which is not a valid value for type 'nsEventStatus' Seems to be closely related to bug 1199885 comment 101.
|
Assignee | |
Comment 1•7 years ago
|
||
Looks to me like nsEventStatus_eIgnore is the way this is done in other parts of the code.
Attachment #8913492 -
Flags: review?(bugmail)
|
||
Updated•7 years ago
|
Attachment #8913492 -
Flags: review?(bugmail) → review+
|
|
||
Updated•7 years ago
|
Assignee: nobody → arthuredelstein
Priority: -- → P3
Whiteboard: [tor] → [tor][gfx-noted]
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/03f60efbf009 Fix ubsan runtime error for nsEventStatus enum. r=kats
Keywords: checkin-needed
|
|
||
Updated•7 years ago
|
status-firefox55:
--- → wontfix
status-firefox56:
--- → wontfix
status-firefox57:
--- → wontfix
status-firefox58:
--- → affected
|
||
Comment 4•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/03f60efbf009
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•