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

Commit 27330416 authored by Howard Ro's avatar Howard Ro
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
parent b32d038f

core/jni/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -232,6 +232,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"
@@ -853,6 +854,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;