Closed
Bug 949360
Opened 9 years ago
Closed 9 years ago
Check for Image AllocateBuffer() failures
Categories
(Core :: Graphics: Layers, defect)
Core
Graphics: Layers
Tracking
()
RESOLVED
FIXED
mozilla29
People
(Reporter: cpeterson, Assigned: cpeterson)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
3.94 KB,
patch
|
nical
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This patch began as a fix for a warning that local variable `status` was unused in release builds: gfx/layers/ipc/SharedPlanarYCbCrImage.cpp:133:8 [-Wunused-variable] unused variable 'status' But then I saw that a couple Image functions did not check whether AllocateBuffer() returned null. This patch checks for AllocateBuffer() returning null and also avoids setting mBufferSize to a non-zero size when mBuffer is null. Green try run: https://tbpl.mozilla.org/?tree=Try&rev=1598ed0935bc
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8346423 -
Flags: review?(nical.bugzilla)
Updated•9 years ago
|
Attachment #8346423 -
Flags: review?(nical.bugzilla) → review+
Comment 4•9 years ago
|
||
Backed out because of mochitest-3 timeouts: http://hg.mozilla.org/integration/mozilla-inbound/rev/7631ae46b853 https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=6d9cf51a012a Note that the WinXP mochitest-3 crashes were probably caused by bug 948777.
Comment 5•9 years ago
|
||
And your name is cleared! Relanded: https://hg.mozilla.org/integration/mozilla-inbound/rev/6f419c7c2873
https://hg.mozilla.org/mozilla-central/rev/6f419c7c2873
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8346423 [details] [diff] [review] handle-AllocateBuffer-failure.patch [Approval Request Comment] Bug caused by (feature/regressing bug #): Not a regression, just some omitted error checks User impact if declined: No big problems. These error checks have been missing for a couple releases without causing any major problems. Testing completed (on m-c, etc.): m-c for one week Risk to taking this patch (and alternatives if risky): Low risk. It just adds a couple error checks for failed allocations of graphics buffer allocation. String or IDL/UUID changes made by this patch: None
Attachment #8346423 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
Attachment #8346423 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/274f1e06b821
You need to log in
before you can comment on or make changes to this bug.
Description
•