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
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]
: Andrew Overholt [:overholt]
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:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

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 User image 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 User image Vicamo Yang [:vicamo][:vyang] 2012-04-24 23:30:57 PDT
Created attachment 618173 [details] [diff] [review]
Fix SMS message.header undefined
Comment 2 User image Philipp von Weitershausen [:philikon] 2012-04-25 19:37:10 PDT
Comment 3 User image Ed Morley [:emorley] 2012-04-26 10:35:37 PDT

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