Closed
Bug 829420
Opened 11 years ago
Closed 11 years ago
[Camera] In GonkNativeWindow::dequeueBuffer, -1 will be used as index to access an Arrary
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mchen, Assigned: mchen)
Details
Attachments
(1 file, 1 obsolete file)
1.35 KB,
patch
|
mchen
:
review+
|
Details | Diff | Splinter Review |
Please refer to link as below for exact code position. http://mxr.mozilla.org/mozilla-central/source/dom/camera/GonkNativeWindow.cpp#228 And to use -1 as an index to access array may causes segfault.
Assignee | ||
Comment 1•11 years ago
|
||
Before use variable - found as an index, to check whether it is small then 0 first.
Assignee: nobody → mchen
Attachment #700860 -
Flags: review?(kchen)
Assignee | ||
Updated•11 years ago
|
Attachment #700860 -
Attachment is patch: true
Comment 2•11 years ago
|
||
Comment on attachment 700860 [details] [diff] [review] Patch v1 Review of attachment 700860 [details] [diff] [review]: ----------------------------------------------------------------- nice catch
Attachment #700860 -
Flags: review?(kchen) → review+
Assignee | ||
Comment 3•11 years ago
|
||
1. Add reviewer. 2. Wait for try. https://tbpl.mozilla.org/?tree=Try&rev=099f53085d4b
Attachment #700860 -
Attachment is obsolete: true
Attachment #700945 -
Flags: review+
Assignee | ||
Comment 4•11 years ago
|
||
All green in try. https://tbpl.mozilla.org/?tree=Try&rev=099f53085d4b
Keywords: checkin-needed
Comment 6•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/3e7dc5bede82
You need to log in
before you can comment on or make changes to this bug.
Description
•