Better zygote process detection

native/jni/include/daemon.h

@@ -79,7 +79,6 @@ void install_apk(const char *apk);
  **************/
 
 void magiskhide_handler(int client);
-void zygote_notify(int client, struct ucred *cred);
 
 /*************
  * Superuser *

native/jni/magiskhide/hide_utils.cpp

@@ -18,7 +18,7 @@
 
 using namespace std;
 
-pthread_t proc_monitor_thread;
+static pthread_t proc_monitor_thread;
 
 static const char *prop_key[] =
 	{ "ro.boot.vbmeta.device_state", "ro.boot.verifiedbootstate", "ro.boot.flash.locked",
@@ -327,4 +327,3 @@ void auto_start_magiskhide() {
 		});
 	}
 }
-

native/jni/magiskhide/magiskhide.h

@@ -37,7 +37,6 @@ void clean_magisk_props();
 void crawl_procfs(const std::function<bool (int)> &fn);
 bool proc_name_match(int pid, const char *name);
 
-extern pthread_t proc_monitor_thread;
 extern bool hide_enabled;
 extern pthread_mutex_t monitor_lock;
 extern std::set<std::pair<std::string, std::string>> hide_set;

native/jni/magiskhide/proc_monitor.cpp

@@ -63,6 +63,22 @@ static inline void lazy_unmount(const char* mountpoint) {
 		LOGD("hide_daemon: Unmounted (%s)\n", mountpoint);
 }
 
+static int parse_ppid(int pid) {
+	char path[32];
+	int ppid;
+
+	sprintf(path, "/proc/%d/stat", pid);
+	FILE *stat = fopen(path, "re");
+	if (stat == nullptr)
+		return -1;
+
+	/* PID COMM STATE PPID ..... */
+	fscanf(stat, "%*d %*s %*c %d", &ppid);
+	fclose(stat);
+
+	return ppid;
+}
+
 static long xptrace(bool log, int request, pid_t pid, void *addr, void *data) {
 	long ret = ptrace(request, pid, addr, data);
 	if (log && ret == -1)
@@ -115,19 +131,26 @@ static bool parse_packages_xml(string_view s) {
 }
 
 static void check_zygote() {
+	int min_zyg = 1;
+	if (access("/system/bin/app_process64", R_OK) == 0)
+		min_zyg = 2;
+	for (bool first = true; zygote_map.size() < min_zyg; first = false) {
+		if (!first)
+			usleep(10000);
 		crawl_procfs([](int pid) -> bool {
 			char buf[512];
 			snprintf(buf, sizeof(buf), "/proc/%d/cmdline", pid);
 			FILE *f = fopen(buf, "re");
 			if (f) {
 				fgets(buf, sizeof(buf), f);
-			if (strncmp(buf, "zygote", 6) == 0)
+				if (strncmp(buf, "zygote", 6) == 0 && parse_ppid(pid) == 1)
 					new_zygote(pid);
 				fclose(f);
 			}
 			return true;
 		});
 	}
+}
 
 void *update_uid_map(void*) {
 	MutexGuard lock(monitor_lock);
@@ -138,7 +161,7 @@ void *update_uid_map(void*) {
 
 /*************************
  * The actual hide daemon
- **************************/
+ *************************/
 
 static void hide_daemon(int pid) {
 	RunFinally fin([=]() -> void {
@@ -208,8 +231,7 @@ static void inotify_event(int) {
 	read(inotify_fd, buf, sizeof(buf));
 	if ((event->mask & IN_CLOSE_WRITE) && strcmp(event->name, "packages.xml") == 0) {
 		LOGD("proc_monitor: /data/system/packages.xml updated\n");
-		check_zygote();
+		new_daemon_thread(update_uid_map);
-		update_uid_map();
 	}
 }