diff options
Diffstat (limited to 'test')
| -rw-r--r-- | test/600-verifier-fails/expected.txt | 1 | ||||
| -rw-r--r-- | test/600-verifier-fails/info.txt | 20 | ||||
| -rw-r--r-- | test/600-verifier-fails/smali/iget.smali | 25 | ||||
| -rw-r--r-- | test/600-verifier-fails/src/Main.java | 3 |
4 files changed, 41 insertions, 8 deletions
diff --git a/test/600-verifier-fails/expected.txt b/test/600-verifier-fails/expected.txt index 010f1b74b2..8399969a2d 100644 --- a/test/600-verifier-fails/expected.txt +++ b/test/600-verifier-fails/expected.txt @@ -1,3 +1,4 @@ passed A passed B passed C +passed D diff --git a/test/600-verifier-fails/info.txt b/test/600-verifier-fails/info.txt index 677b4775d5..f77de05ac7 100644 --- a/test/600-verifier-fails/info.txt +++ b/test/600-verifier-fails/info.txt @@ -1,14 +1,18 @@ The situations in these tests were discovered by running the mutating dexfuzz on the DEX files of fuzzingly random generated Java test. -(1) b/28908555: - soft verification fail (on the final field modification) should - not hide the hard verification fail (on the type mismatch) to +(A) b/28908555: + soft verification failure (on the final field modification) should + not hide the hard verification failure (on the type mismatch) to avoid compiler crash later on -(2) b/29070461: - hard failure (not calling super in constructor) should bail - immediately and not allow soft fails to pile up behind it to - avoid fatal message later on -(3) b/29068831: +(B) b/29070461: + hard verification failure (not calling super in constructor) should + bail immediately and not allow soft verification failures to pile up + behind it to avoid fatal message later on +(C) b/29068831: access validation should occur prior to null reference check +(D) b/29126870: + soft verification failure (cannot access) should not hide the hard + verification failure (non-reference type) to avoid a compiler crash + later on diff --git a/test/600-verifier-fails/smali/iget.smali b/test/600-verifier-fails/smali/iget.smali new file mode 100644 index 0000000000..5c045e6b76 --- /dev/null +++ b/test/600-verifier-fails/smali/iget.smali @@ -0,0 +1,25 @@ +# +# Copyright (C) 2016 The Android Open Source Project +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +.class public LD; +.super Ljava/lang/Object; + +.method public constructor <init>()V + .registers 2 + invoke-direct {v1}, Ljava/lang/Object;-><init>()V + const v0, 2 + iget v1, v0, LMain;->privateField:I + return-void +.end method diff --git a/test/600-verifier-fails/src/Main.java b/test/600-verifier-fails/src/Main.java index 0a8c5a179d..a6a07fda79 100644 --- a/test/600-verifier-fails/src/Main.java +++ b/test/600-verifier-fails/src/Main.java @@ -20,6 +20,8 @@ public class Main { private static String staticPrivateField = null; + private int privateField = 0; + private static void test(String name) throws Exception { try { Class<?> a = Class.forName(name); @@ -33,5 +35,6 @@ public class Main { test("A"); test("B"); test("C"); + test("D"); } } |