Last Comment Bug 748667 - B2G SMS: can't receive normal SMS, message.header is undefined
: B2G SMS: can't receive normal SMS, message.header is undefined
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: unspecified
: ARM Gonk (Firefox OS)
: -- normal (vote)
: mozilla15
Assigned To: Vicamo Yang [:vicamo][:vyang]
:
Mentors:
Depends on:
Blocks: b2g-sms
  Show dependency treegraph
 
Reported: 2012-04-24 23:21 PDT by Vicamo Yang [:vicamo][:vyang]
Modified: 2012-04-26 10:35 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Fix SMS message.header undefined (1.06 KB, patch)
2012-04-24 23:30 PDT, Vicamo Yang [:vicamo][:vyang]
philipp: review+
Details | Diff | Splinter Review

Description Vicamo Yang [:vicamo][:vyang] 2012-04-24 23:21:01 PDT
Bug 738132 introduces support for application port addressing but didn't check existence of `message.header` before looking into its properties. This causes an error as:

  E/GeckoConsole( 2922): [JavaScript Error: "message.header is undefined" {file:
  "jar:file:///system/b2g/omni.ja!/components/RadioInterfaceLayer.js" line: 491}]
Comment 1 Vicamo Yang [:vicamo][:vyang] 2012-04-24 23:30:57 PDT
Created attachment 618173 [details] [diff] [review]
Fix SMS message.header undefined
Comment 2 Philipp von Weitershausen [:philikon] 2012-04-25 19:37:10 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/019380fcd070
Comment 3 Ed Morley [:emorley] 2012-04-26 10:35:37 PDT
https://hg.mozilla.org/mozilla-central/rev/019380fcd070

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