commit 4db659e8d9e51aa5d68884e051e8135070efd42c
author Michael Hoisie <hoisie@google.com> Sat May 16 20:13:29 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Sat May 16 20:13:29 2020 +0000
tree 034daa1f01c0f0fc1084c567ca79994842f01abc
parent beee98a4753c8d95dfc1d8df9382533126ecb0df
parent 9b5bf0f42d57387a3451aa9eb9bfb1075fcb91a4

Merge "Add null check for fgets in BufferQueueCore" into rvc-dev am: 9b5bf0f42d

Change-Id: I6bd959e885eb0a8acbceb80e604130c558227978

libs/gui/BufferQueueCore.cpp

diff --git a/libs/gui/BufferQueueCore.cpp b/libs/gui/BufferQueueCore.cpp
index 8d80833..5023b6b 100644
--- a/libs/gui/BufferQueueCore.cpp
+++ b/libs/gui/BufferQueueCore.cpp
@@ -75,10 +75,12 @@
     if (NULL != fp) {
         const size_t size = 64;
         char proc_name[size];
-        fgets(proc_name, size, fp);
+        char* result = fgets(proc_name, size, fp);
         fclose(fp);
-        name = proc_name;
-        return NO_ERROR;
+        if (result != nullptr) {
+            name = proc_name;
+            return NO_ERROR;
+        }
     }
     return INVALID_OPERATION;
 }