commit 10cd48da1d1c5c63e9e890c872aecc7160122f71
author Matthew Bouyack <mbouyack@google.com> Tue Sep 20 19:08:42 2016 -0700
committer Tao Bao <tbao@google.com> Sun Nov 20 22:38:37 2016 -0800
tree 80b6f372d5756af5ba4ee4872a3464ea4a697043
parent 1d77c93bcc446971d700ca42bfc14cde207c1795

Fail gracefully when we fail to fork the update binary

This change was original made in cw-f-dev but caused failures in
nyc-mr1-dev-plus-aosp due to lack of support for 'LOGE'

This version of the change uses the new 'LOG(ERROR)' style logging
instead.

See bug b/31395655
Test: attempt a memory intensive incremental OTA on a low-memory device

Change-Id: Ia87d989a66b0ce3f48e862abf9b9d6943f70e554
(cherry picked from commit c8db4817809e163d887f7955a03ad0f97159f12b)

install.cpp

diff --git a/install.cpp b/install.cpp
index 98f1e3f..f124a26 100644
--- a/install.cpp
+++ b/install.cpp
@@ -373,6 +373,14 @@
     }
 
     pid_t pid = fork();
+
+    if (pid == -1) {
+        close(pipefd[0]);
+        close(pipefd[1]);
+        PLOG(ERROR) << "Failed to fork update binary";
+        return INSTALL_ERROR;
+    }
+
     if (pid == 0) {
         umask(022);
         close(pipefd[0]);