diff options
author | Simon Wilson <simonwilson@google.com> | 2011-03-07 15:42:59 -0800 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2011-03-07 15:42:59 -0800 |
commit | 10993b2d1e3614073970aad119ffb2cc9bc8f73f (patch) | |
tree | d03f254077b936bdc90dd524e59735f9747fc97b | |
parent | b1be086e1d2f15c858b4aa7f40d49667b3e26d3a (diff) | |
parent | 6b6869d5c1a203e79dc272f2f0e53c184f44f24a (diff) | |
download | frameworks_base-10993b2d1e3614073970aad119ffb2cc9bc8f73f.zip frameworks_base-10993b2d1e3614073970aad119ffb2cc9bc8f73f.tar.gz frameworks_base-10993b2d1e3614073970aad119ffb2cc9bc8f73f.tar.bz2 |
am 6b6869d5: Merge "mms: mms wap push length check before pdu processing." into gingerbread
* commit '6b6869d5c1a203e79dc272f2f0e53c184f44f24a':
mms: mms wap push length check before pdu processing.
-rwxr-xr-x | telephony/java/com/android/internal/telephony/cdma/sms/BearerData.java | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/telephony/java/com/android/internal/telephony/cdma/sms/BearerData.java b/telephony/java/com/android/internal/telephony/cdma/sms/BearerData.java index e95ec44..7bc48ca 100755 --- a/telephony/java/com/android/internal/telephony/cdma/sms/BearerData.java +++ b/telephony/java/com/android/internal/telephony/cdma/sms/BearerData.java @@ -1015,17 +1015,17 @@ public final class BearerData { boolean decodingtypeUTF8 = Resources.getSystem() .getBoolean(com.android.internal.R.bool.config_sms_utf8_support); - if (!decodingtypeUTF8) { - // Strip off any padding bytes, meaning any differences between the length of the - // array and the target length specified by numFields. This is to avoid any - // confusion by code elsewhere that only considers the payload array length. - byte[] payload = new byte[userData.numFields]; - int copyLen = userData.numFields < userData.payload.length - ? userData.numFields : userData.payload.length; + // Strip off any padding bytes, meaning any differences between the length of the + // array and the target length specified by numFields. This is to avoid any + // confusion by code elsewhere that only considers the payload array length. + byte[] payload = new byte[userData.numFields]; + int copyLen = userData.numFields < userData.payload.length + ? userData.numFields : userData.payload.length; - System.arraycopy(userData.payload, 0, payload, 0, copyLen); - userData.payload = payload; + System.arraycopy(userData.payload, 0, payload, 0, copyLen); + userData.payload = payload; + if (!decodingtypeUTF8) { // There are many devices in the market that send 8bit text sms (latin encoded) as // octet encoded. userData.payloadStr = decodeLatin(userData.payload, offset, userData.numFields); |