OSDN Git Service

Fix data type of RIL_UNSOL_RESPONSE_NEW_BROADCAST_SMS
authorHenrik Hall <henrik.hall@sonyericsson.com>
Wed, 24 Feb 2010 11:53:13 +0000 (12:53 +0100)
committerChih-Wei Huang <cwhuang@linux.org.tw>
Fri, 15 Apr 2011 10:24:31 +0000 (18:24 +0800)
Fixing incorrect data type of RIL_UNSOL_RESPONSE_NEW_BROADCAST_SMS
in Android RIL.

The Android RIL specifies that the unsolicited response signal
RIL_UNSOL_RESPONSE_NEW_BROADCAST_SMS should contain a raw byte array
(ril.h). However, in the implementation of the Java RIL and the C++
RIL daemon, the data is unpacked as a string. This causes data
delivered from the network stack to be garbled.

Changing so that both the C++ RIL deamon in platform/hardware/ril and
the Java RIL in frameworks/base follows the specification.

Change-Id: Ifa1b9907934256a46cba2b77a560873199e761dd

libril/ril_unsol_commands.h

index 3660702..6eddd0f 100644 (file)
@@ -35,7 +35,7 @@
     {RIL_UNSOL_CALL_RING, responseCallRing, WAKE_PARTIAL},
     {RIL_UNSOL_RESPONSE_SIM_STATUS_CHANGED, responseVoid, WAKE_PARTIAL},
     {RIL_UNSOL_RESPONSE_CDMA_NEW_SMS, responseCdmaSms, WAKE_PARTIAL},
-    {RIL_UNSOL_RESPONSE_NEW_BROADCAST_SMS, responseString, WAKE_PARTIAL},
+    {RIL_UNSOL_RESPONSE_NEW_BROADCAST_SMS, responseRaw, WAKE_PARTIAL},
     {RIL_UNSOL_CDMA_RUIM_SMS_STORAGE_FULL, responseVoid, WAKE_PARTIAL},
     {RIL_UNSOL_RESTRICTED_STATE_CHANGED, responseInts, WAKE_PARTIAL},
     {RIL_UNSOL_ENTER_EMERGENCY_CALLBACK_MODE, responseVoid, WAKE_PARTIAL},