commit 2c20bbf9c81bf44b56af359e29c483f205c81715
author Steven Moreland <smoreland@google.com> Wed Oct 30 15:18:58 2019 -0700
committer Steven Moreland <smoreland@google.com> Wed Oct 30 15:19:39 2019 -0700
tree 7d52a0fd45caf312e26b06f5be5033ee48af5fcd
parent df9ad97a5300ad5d322b04c6c3d4f526459f32bf

Better error message for bad array write.

Noticed this didn't match the check.

Bug: 143556646
Test: N/A
Change-Id: If6b1a63844a7ffbd325f20ed00852f83d3e3a9c1

libs/binder/ndk/parcel.cpp

diff --git a/libs/binder/ndk/parcel.cpp b/libs/binder/ndk/parcel.cpp
index ae2276e..f18e118 100644
--- a/libs/binder/ndk/parcel.cpp
+++ b/libs/binder/ndk/parcel.cpp
@@ -50,7 +50,7 @@
     if (length < -1) return STATUS_BAD_VALUE;
 
     if (!isNullArray && length < 0) {
-        LOG(ERROR) << __func__ << ": null array must be used with length == -1.";
+        LOG(ERROR) << __func__ << ": non-null array but length is " << length;
         return STATUS_BAD_VALUE;
     }
     if (isNullArray && length > 0) {