sam.hua Reporter
	Description • 10 years ago

When receive CBS messages, ril_worker.js throws the error:
06-06 08:27:15.150   108   506 I Gecko   : RIL Worker: [0] Handling parcel as UNSOLICITED_RESPONSE_NEW_BROADCAST_SMS
06-06 08:27:15.150   108   506 I Gecko   : RIL Worker: Parcel handling threw TypeError: original is null
06-06 08:27:15.150   108   506 I Gecko   : RilObject.prototype._processReceivedSmsCbPage@resource://gre/modules/ril_worker.js:4767:1
06-06 08:27:15.150   108   506 I Gecko   : UNSOLICITED_RESPONSE_NEW_BROADCAST_SMS@resource://gre/modules/ril_worker.js:6587:3
06-06 08:27:15.150   108   506 I Gecko   : RilObject.prototype.handleParcel@resource://gre/modules/ril_worker.js:5182:7
06-06 08:27:15.150   108   506 I Gecko   : BufObject.prototype.processParcel@resource://gre/modules/ril_worker.js:159:5
06-06 08:27:15.150   108   506 I Gecko   : require._tmpModules[":resource://gre/modules/workers/worker_buf.js"]/Buf.processIncoming@blob:052f97d2-e4af-4980-9f25-a823e801d1f1:536:9
06-06 08:27:15.150   108   506 I Gecko   : ContextPool.handleRilMessage@resource://gre/modules/ril_worker.js:14612:5
06-06 08:27:15.150   108   506 I Gecko   : onRILMessage@resource://gre/modules/ril_worker.js:14659:3
06-06 08:27:15.150   108   506 I Gecko   : 
06-06 08:27:15.150   108   506 I Gecko   : RIL Worker: Parcel handler didn't consume whole parcel, 328 bytes left over

	sam.hua Reporter
	Comment 1 • 10 years ago

main log

	sam.hua Reporter
	Comment 2 • 10 years ago

radio.log

	thomas tsai
	Comment 3 • 10 years ago

Hi Hsinyi, can you help?

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 4 • 10 years ago

It seems that the PDU in UNSOLICITED_RESPONSE_NEW_BROADCAST_SMS is packed in UMTS format (> 88 octets) defined in 9.4.2 UMTS of 23.041 [1] which is not supported yet in our implementation[2]. :(

[1] http://www.etsi.org/deliver/etsi_ts/123000_123099/123041/11.06.00_60/ts_123041v110600p.pdf
[2] http://hg.mozilla.org/releases/mozilla-b2g30_v1_4/file/30224c7f5e58/dom/system/gonk/ril_worker.js#l8333

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 5 • 10 years ago

Thanks, Bevis :)

	sam.hua Reporter
	Comment 6 • 10 years ago

could us try the patch of 813893?

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 7 • 10 years ago

(In reply to sam.hua from comment #6)
> could us try the patch of 813893?

Sure, but I am not sure if it's workable yet and re-base might be needed since that patch was provided about 1.5 yr ago.

	sam.hua Reporter
	Comment 8 • 10 years ago

yes, it failed.
Hi htsai,

when it could be fixed?
Thanks!

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 9 • 10 years ago

Hi Sam, 
Is this certification blocker?

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 10 • 10 years ago

ni? Ivan to decide how we should handle remaining Dolphin bugs.

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 11 • 10 years ago

PM is helping prioritize this bug. Cancelling NI.

	bhavana bajaj [:bajaj]
	Comment 12 • 10 years ago

If this needs Bug 813893, looks like a new feature request, we should not be adding new features at this point.

	Ivan Tsay (:ITsay)
	Comment 13 • 10 years ago

(In reply to bhavana bajaj [:bajaj] from comment #12)
> If this needs Bug 813893, looks like a new feature request, we should not be
> adding new features at this point.

Wesley,

Could you help to confirm if this is bug fixing or feature work? We won't take features for v1.4 now.

	Ivan Tsay (:ITsay)
	Comment 14 • 10 years ago

meant to ni? Wesley.

	sam.hua Reporter
	Comment 15 • 10 years ago

the new patch can work in v1.4 (dolphin)

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 16 • 10 years ago

yes, this is feature.

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 17 • 10 years ago

This should be a duplicate of bug 813893. Feel free to reopen if not the case.