commit 2407ef209d3b53468477de87e4a17f671baa52ac
author Tri Vo <trong@google.com> Sun Jul 23 15:45:09 2017 -0700
committer Tri Vo <trong@google.com> Wed Nov 01 14:12:21 2017 -0700
tree 80f98b963adfa5d694c469d5579a334c67ba0568
parent 95dd6559f0c0e886678e8ce8b95c616a8b5f9f37

Ignore nested interface is nullptr.

If a function call returns a pointer to a HIDL proxy and that pointer is
nullptr, ignore do not register it as an opened interface.

Test: SANITIZE_TARGET="address coverage" make vts -j64 && vts-tradefed
run commandAndExit vts --skip-all-system-status-check --primary-abi-only
--skip-preconditions -l VERBOSE --module
VtsHalAudioEffectV2_0IfaceFuzzer
Merged-In: I392618ae624b0d450104416dba366b2aefb348e0
Change-Id: I392618ae624b0d450104416dba366b2aefb348e0

(cherry picked from commit d4168dc56e524668218c04a079ee7a800a8368bd)

iface_fuzzer/ProtoFuzzerRunner.cpp

diff --git a/iface_fuzzer/ProtoFuzzerRunner.cpp b/iface_fuzzer/ProtoFuzzerRunner.cpp
index 3f3422f..7c9ea84 100644
--- a/iface_fuzzer/ProtoFuzzerRunner.cpp
+++ b/iface_fuzzer/ProtoFuzzerRunner.cpp
@@ -201,7 +201,10 @@
 
 void ProtoFuzzerRunner::ProcessReturnValue(const FuncSpec &result) {
   for (const auto &var : result.return_type_hidl()) {
-    if (var.has_hidl_interface_pointer() && var.has_predefined_type()) {
+    // If result contains a pointer to an interface, register it in
+    // opened_ifaces_ table. That pointer must not be a nullptr.
+    if (var.has_hidl_interface_pointer() && var.hidl_interface_pointer() &&
+        var.has_predefined_type()) {
       uint64_t hidl_service = var.hidl_interface_pointer();
       string type = var.predefined_type();
       string iface_name = StripNamespace(type);