Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 65e48ec1 authored by Howard Ro's avatar Howard Ro Committed by Chris Wailes
Browse files

Close statsd log socket before zygote fork(). 

MetricsLogger will start to write to statsd instead of logd. Zygote uses
MetricsLogger too. So we need to close statsd socket before zygote fork() is
called (see b/30963384 for context).

Bug: 110537511
Test: Manually verified that it successfully boots with no crash with
ag/5112579 and ag/5169664 (together)

Change-Id: Id550fbd5ac2f6ca47d3293c12fb537fafa3f33a0
Merged-In: Id550fbd5ac2f6ca47d3293c12fb537fafa3f33a0
parent fc406201

core/jni/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -229,6 +229,7 @@ cc_library_shared { 
        "libseccomp_policy",

        "libgrallocusage",

        "libscrypt_static",

        "libstatssocket",

    ],



    shared_libs: [

core/jni/com_android_internal_os_Zygote.cpp

+2 −0
Original line number Diff line number Diff line
@@ -56,6 +56,7 @@ 
#include <utils/String8.h>

#include <selinux/android.h>

#include <seccomp_policy.h>

#include <stats_event_list.h>

#include <processgroup/processgroup.h>



#include "core_jni_helpers.h"
@@ -788,6 +789,7 @@ static pid_t ForkCommon(JNIEnv* env, jstring java_se_name, bool is_system_server 
  // Close any logging related FDs before we start evaluating the list of

  // file descriptors.

  __android_log_close();

  stats_log_close();



  std::string error_msg;