commit f4a342abd2959904caa18b5a0671e8107581ff24
author Mikhail Naganov <mnaganov@google.com> Tue Dec 04 08:55:41 2018 -0800
committer Mikhail Naganov <mnaganov@google.com> Tue Dec 04 08:55:41 2018 -0800
tree bd8a54b016803bbc5e08a47f250f093e7036d0f4
parent e8c8b4359835a12a3d1bf84f281c36bce031bb33

Fix potential nullptr dereference in RecordThread::dumpInternals

The code introduced in ag/5195499 can deref a nullptr.

Bug: 117299316
Test: make
Change-Id: I6d38f769e583a11d09c137d7b7a1ffd0b547f161

services/audioflinger/Threads.cpp

diff --git a/services/audioflinger/Threads.cpp b/services/audioflinger/Threads.cpp
index 46e40c7..45d3a06 100644
--- a/services/audioflinger/Threads.cpp
+++ b/services/audioflinger/Threads.cpp
@@ -7562,7 +7562,7 @@
         (void)input->stream->dump(fd);
     }
 
-    const double latencyMs = RecordTrack::checkServerLatencySupported(mFormat, input->flags)
+    const double latencyMs = RecordTrack::checkServerLatencySupported(mFormat, flags)
             ? - mTimestamp.getOutputServerLatencyMs(mSampleRate) : 0.;
     if (latencyMs != 0.) {
         dprintf(fd, "  NormalRecord latency ms: %.2lf\n", latencyMs);