summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Wilson <simonwilson@google.com>2011-03-07 15:42:59 -0800
committerAndroid Git Automerger <android-git-automerger@android.com>2011-03-07 15:42:59 -0800
commit10993b2d1e3614073970aad119ffb2cc9bc8f73f (patch)
treed03f254077b936bdc90dd524e59735f9747fc97b
parentb1be086e1d2f15c858b4aa7f40d49667b3e26d3a (diff)
parent6b6869d5c1a203e79dc272f2f0e53c184f44f24a (diff)
downloadframeworks_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-xtelephony/java/com/android/internal/telephony/cdma/sms/BearerData.java18
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);