diff options
| author | Jake Hamby <jhamby@google.com> | 2012-08-28 20:11:34 -0700 |
|---|---|---|
| committer | Jake Hamby <jhamby@google.com> | 2012-08-29 16:36:22 -0700 |
| commit | b35e4d778a95d89c9c9317de0f93f51faab5bbac (patch) | |
| tree | 92e8d7412b1ce576f43e865a2d2ad96eaf1476c2 /docs/html/training/basics/activity-lifecycle | |
| parent | cf9e9d627367d310fb910a052dda1e020748a94c (diff) | |
| download | frameworks_base-b35e4d778a95d89c9c9317de0f93f51faab5bbac.zip frameworks_base-b35e4d778a95d89c9c9317de0f93f51faab5bbac.tar.gz frameworks_base-b35e4d778a95d89c9c9317de0f93f51faab5bbac.tar.bz2 | |
Fix CDMA decoding of multipart UTF-16 SMS messages.
Recent changes to support CMAS over CDMA introduced a bug causing
an exception to be thrown when decoding multipart UTF-16 encoded
messages. This change fixes the exception by correctly subtracting
the header size from the number of bytes to decode. It also adds
more robust error handling to try to decode the maximum length
possible instead of throwing an exception if the length is still
larger than the user data length after subtracting the header.
This also fixes a bug in the encoder, which was padding the
UTF-16 user data to 16-bit alignment, which is incorrect (should
be padded to an 8-bit boundary). The code happened to work because
we always generated a UDH that was an even number of bytes
(including length) so the padding was a no-op. The decoder works
correctly.
Bug: 6939151
Change-Id: Iba9e7156bd7df94e972963959a7ce1c78464f7f5
Diffstat (limited to 'docs/html/training/basics/activity-lifecycle')
0 files changed, 0 insertions, 0 deletions