aboutsummaryrefslogtreecommitdiff
path: root/src/main/java/com/google/android
diff options
context:
space:
mode:
authorryancllin <59638643+ryancllin@users.noreply.github.com>2020-01-10 01:05:55 +0800
committerAng Li <angli@google.com>2020-01-09 09:05:55 -0800
commit58820984f8e79b250bc63d6b47b5f2910876a631 (patch)
treec7e4166b667d6cf849ad30c353f51a560b481477 /src/main/java/com/google/android
parent13af002cd91ac7e06d064bb3a81ca2d00010c313 (diff)
downloadmobly-bundled-snippets-58820984f8e79b250bc63d6b47b5f2910876a631.tar.gz
Fix exception in "trimQuotationMarks()" method (#125)
Add a string length check on the given param to avoid StringIndexOutOfBoundsException.
Diffstat (limited to 'src/main/java/com/google/android')
-rw-r--r--src/main/java/com/google/android/mobly/snippet/bundled/utils/JsonSerializer.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main/java/com/google/android/mobly/snippet/bundled/utils/JsonSerializer.java b/src/main/java/com/google/android/mobly/snippet/bundled/utils/JsonSerializer.java
index ef7f7d7..82e1e4f 100644
--- a/src/main/java/com/google/android/mobly/snippet/bundled/utils/JsonSerializer.java
+++ b/src/main/java/com/google/android/mobly/snippet/bundled/utils/JsonSerializer.java
@@ -62,7 +62,8 @@ public class JsonSerializer {
*/
public static String trimQuotationMarks(String originalString) {
String result = originalString;
- if (originalString.charAt(0) == '"'
+ if (originalString.length() > 2
+ && originalString.charAt(0) == '"'
&& originalString.charAt(originalString.length() - 1) == '"') {
result = originalString.substring(1, originalString.length() - 1);
}