Last Comment Bug 757512 - B2G RIL: SIM Card Lock DOM Requests don't properly clean up
: B2G RIL: SIM Card Lock DOM Requests don't properly clean up
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: unspecified
: ARM Gonk (Firefox OS)
-- normal (vote)
: mozilla15
Assigned To: Yoshi Huang[:allstars.chh]
: Andrew Overholt [:overholt]
Depends on:
Blocks: 731786
  Show dependency treegraph
Reported: 2012-05-22 10:26 PDT by Marshall Culpepper [:marshall_law]
Modified: 2012-05-24 09:19 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (1.47 KB, patch)
2012-05-22 20:40 PDT, Yoshi Huang[:allstars.chh]
philipp: review+
Details | Diff | Splinter Review

Description User image Marshall Culpepper [:marshall_law] 2012-05-22 10:26:46 PDT
The DOMRequests used to provide callbacks for the nsIDOMMobleConnection.getCardLock/setCardLock/unlockCardLock are not currently cleaning up the allocated DOM requests when status is sent back in ril_worker.js. 

The two examples I found were in dom/system/gonk/RILContentHelper.js (lines 317/325ish), where the use of this.getRequest(requestId) could be simply changed to this.takeRequest(requestId)
Comment 1 User image Yoshi Huang[:allstars.chh] 2012-05-22 20:40:40 PDT
Created attachment 626310 [details] [diff] [review]
Comment 2 User image Yoshi Huang[:allstars.chh] 2012-05-22 20:41:19 PDT
Comment on attachment 626310 [details] [diff] [review]

Thanks for Marshall to point this out.
Comment 3 User image Philipp von Weitershausen [:philikon] 2012-05-23 22:25:39 PDT
Comment 4 User image Ed Morley [:emorley] 2012-05-24 09:19:55 PDT

Note You need to log in before you can comment on or make changes to this bug.