| commit | b9d683275284471d594a8f2f89e8738fa07ab9f3 | [log] [tgz] |
|---|---|---|
| author | Victor Hsieh <victorhsieh@google.com> | Fri Feb 08 09:56:16 2019 -0800 |
| committer | Victor Hsieh <victorhsieh@google.com> | Fri Feb 08 10:38:54 2019 -0800 |
| tree | 7fc1130664b684b4c6fff1e4c3a210f40fd88412 | |
| parent | 79974722c4bcb71a6235cf1a1072a67b746afd9c [diff] [blame] |
Fix crash by wrong assertion check The original destructor didn't actually run. But since ag/6239075 fixed the memory leak, this starts to crash. Test: atest ApkVerityTest (not yet submitted) Bug: 112037636 Change-Id: Ie7fe3eaf9e1ff1b374243b1d78417423ffba0049
diff --git a/services/core/jni/com_android_server_security_VerityUtils.cpp b/services/core/jni/com_android_server_security_VerityUtils.cpp index bf96f9a..6cd9f2c 100644 --- a/services/core/jni/com_android_server_security_VerityUtils.cpp +++ b/services/core/jni/com_android_server_security_VerityUtils.cpp
@@ -122,7 +122,7 @@ } ~JavaByteArrayHolder() { - LOG_ALWAYS_FATAL_IF(mElements == nullptr, "Elements are not released"); + LOG_ALWAYS_FATAL_IF(mElements != nullptr, "Elements are not released"); } private: