Merge pull request #123 from en4bz/fix-clone3

Fix clone3 crash in glibc 2.34+

Author: en4bz

src/intercept.c

@@ -52,6 +52,7 @@
 #include <sys/mman.h>
 #include <stdarg.h>
 #include <sys/auxv.h>
+#include <linux/sched.h>
 
 #include "intercept.h"
 #include "intercept_log.h"
@@ -675,9 +676,17 @@ intercept_routine(struct context *context)
 		 * the clone_child_intercept_routine instead, executing
 		 * it on the new child threads stack, then returns to libc.
 		 */
-		if (desc.nr == SYS_clone && desc.args[1] != 0)
+		if (desc.nr == SYS_clone && desc.args[1] != 0) {
 			return (struct wrapper_ret){
 				.rax = context->rax, .rdx = 2 };
+		}
+#ifdef SYS_clone3
+		else if (desc.nr == SYS_clone3 &&
+			((struct clone_args *)desc.args[0])->stack != 0) {
+			return (struct wrapper_ret){
+				.rax = context->rax, .rdx = 2 };
+		}
+#endif
 		else
 			result = syscall_no_intercept(desc.nr,
 					desc.args[0],

test/hook_test_clone_preload.c

@@ -42,7 +42,7 @@
 #include <string.h>
 #include <unistd.h>
 #include <syscall.h>
-
+#include <linux/sched.h>
 #include "libsyscall_intercept_hook_point.h"
 
 static int hook_counter;
@@ -65,6 +65,10 @@ hook(long syscall_number,
 
 	if (syscall_number == SYS_clone)
 		hook_counter++;
+#ifdef SYS_clone3
+	if (syscall_number == SYS_clone3)
+		hook_counter++;
+#endif
 
 	return 1;
 }

test/test_clone_thread_preload.c

@@ -47,6 +47,7 @@
 #include <assert.h>
 #include <syscall.h>
 #include <stdio.h>
+#include <linux/sched.h>
 
 static long flags = -1;
 
@@ -80,9 +81,15 @@ hook(long syscall_number,
 	 * therefore the return value (the child's pid) can not be observed,
 	 * or modified.
 	 */
-	if (syscall_number == SYS_clone && (arg1 != 0))
+	if (syscall_number == SYS_clone && (arg1 != 0)) {
 		flags = arg0;
-
+	}
+#ifdef SYS_clone3
+	if (syscall_number == SYS_clone3 &&
+			((struct clone_args *)arg0)->stack != 0) {
+		flags = arg0;
+	}
+#endif
 	return 1;
 }