Closed
Bug 1056625
Opened 10 years ago
Closed 10 years ago
[Settings] crash when tapping on either SIM in SIM settings (of Messaging Settings) if the SIM pin is locked
Categories
(Firefox OS Graveyard :: RIL, defect)
Tracking
(b2g-v2.0 unaffected, b2g-v2.1 affected)
VERIFIED
FIXED
2.1 S3 (29aug)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | unaffected |
b2g-v2.1 | --- | affected |
People
(Reporter: ericcc, Assigned: bevis)
Details
Attachments
(1 file)
### STR
1. SIM pin turn ON.
2. Reboot.
3. Skip SIM pin on boot.
4. 'Settings' -> 'Messaging Settings' -> 'SIM settings'.
5. Tap the one with SIM pin locked.
### Actual
Settings crashes
### Expected
No crash
### Version
Gaia 3584b2723412ed3299c6761f465885d80651c87e
Gecko https://hg.mozilla.org/mozilla-central/rev/e7806c9c83f3
BuildID 20140820160201
Version 34.0a1
ro.build.version.incremental=110
ro.build.date=Fri Jun 27 15:57:58 CST 2014
B1TC00011230
### Last Good Build
Gaia aa8aace12d65956dd9525da5dac66e0d3b28597f
Gecko https://hg.mozilla.org/mozilla-central/rev/37ac55a26014
BuildID 20140817160201
Version 34.0a1
ro.build.version.incremental=eng.ericcc.20140815.103628
ro.build.date=Fri Aug 15 10:36:36 CST 2014
Reporter | ||
Comment 1•10 years ago
|
||
[Blocking Requested - why for this release]: Crash
blocking-b2g: --- → 2.1?
Reporter | ||
Updated•10 years ago
|
QA Whiteboard: [COM=Gaia::Settings]
Comment 2•10 years ago
|
||
When calling `var req = mobileMessage.getSmscAddress(this.index);` in locked simcard, then it makes the app crash.
Eric, do you know who should we ni? in Gecko for this part ?!
Comment 3•10 years ago
|
||
(In reply to EJ Chen [:eragonj][:小龍哥] from comment #2)
> When calling `var req = mobileMessage.getSmscAddress(this.index);` in locked
> simcard, then it makes the app crash.
>
> Eric, do you know who should we ni? in Gecko for this part ?!
Our ril team will take a look. Get back to you if we have new findings :)
Assignee | ||
Comment 4•10 years ago
|
||
Root cause found:
1. In nsIMobileMessageCallback.notifyGetSmscAddressFailed(in long error), an XXX_ERROR defined in nsIMobileMessageCallback is expected. [1]
2. However, in RadioInterfaceLayer, we pass the response.errorMsg directly without converting it into corresponding errorCode. [2]
3. Hence, a MOZ_CRASH will be risen while ConvertErrorCodeToErrorString() in MobileMessageCallback.cpp . [3]
A test case for failure scenario is required to cover this regression.
[1] http://dxr.mozilla.org/mozilla-central/source/dom/mobilemessage/interfaces/nsIMobileMessageCallback.idl#56
[2] http://dxr.mozilla.org/mozilla-central/source/dom/system/gonk/RadioInterfaceLayer.js#4252
[3] http://dxr.mozilla.org/mozilla-central/source/dom/mobilemessage/src/MobileMessageCallback.cpp#64
Assignee: nobody → btseng
Component: Gaia::Settings → RIL
Assignee | ||
Comment 5•10 years ago
|
||
a MOZ_CRASH at ConvertErrorCodeToErrorString() in MobileMessageCallback.cpp because an invalid errorCode was provided in RadioInterfaceLayer.
This patch is to reply nsIMobileMessageCallback.NOT_FOUND_ERROR when failure to getSmscAddress().
Hi Vicamo,
May I have your review for the quick fix of this crash?
Thanks!
Attachment #8477257 -
Flags: review?(vyang)
Comment 6•10 years ago
|
||
Comment on attachment 8477257 [details] [diff] [review]
Reply nsIMobileMessageCallback.NOT_FOUND_ERROR instead of response.errorMsg from ril_worker when failed to getSmscAddress().
Review of attachment 8477257 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you for this quick fix!
Attachment #8477257 -
Flags: review?(vyang) → review+
Assignee | ||
Comment 7•10 years ago
|
||
update try srever result:
https://tbpl.mozilla.org/?tree=Try&rev=495eb6ce2a52
Keywords: checkin-needed
Comment 8•10 years ago
|
||
Keywords: checkin-needed
Comment 9•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S3 (29aug)
Updated•10 years ago
|
blocking-b2g: 2.1? → ---
Reporter | ||
Comment 10•10 years ago
|
||
Gaia-Rev b542080231bb9cdbdb282ab926965ad3f91f7460
Gecko-Rev https://hg.mozilla.org/releases/mozilla-aurora/rev/d614ae398c03
Build-ID 20140923155123
Version 34.0a2
Device-Name flame
FW-Release 4.4.2
FW-Incremental 27
FW-Date Thu Sep 4 14:59:02 CST 2014
Bootloader L1TC10011800
Status: RESOLVED → VERIFIED
QA Whiteboard: [COM=Gaia::Settings] → [COM=RIL]
You need to log in
before you can comment on or make changes to this bug.
Description
•