File linux-user-add-binfmt-wrapper-for-argv-0.patch of Package qemu

Overview Repositories Revisions Requests Users Attributes Meta

File linux-user-add-binfmt-wrapper-for-argv-0.patch of Package qemu (Revision 43909338b979fda35565a21db433bd85)

Currently displaying revision 43909338b979fda35565a21db433bd85 , Show latest

From: Alexander Graf <agraf@suse.de>
Date: Fri, 30 Sep 2011 19:40:36 +0200
Subject: linux-user: add binfmt wrapper for argv[0] handling
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When using qemu's linux-user binaries through binfmt, argv[0] gets lost
along the execution because qemu only gets passed in the full file name
to the executable while argv[0] can be something completely different.

This breaks in some subtile situations, such as the grep and make test
suites.

This patch adds a wrapper binary called qemu-$TARGET-binfmt that can be
used with binfmt's P flag which passes the full path _and_ argv[0] to
the binfmt handler.

The binary would be smart enough to be versatile and only exist in the
system once, creating the qemu binary path names from its own argv[0].
However, this seemed like it didn't fit the make system too well, so
we're currently creating a new binary for each target archictecture.

CC: Reinhard Max <max@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
[AF: Rebased onto new Makefile infrastructure, twice]
[AF: Updated for aarch64 for v2.0.0-rc1]
[AF: Rebased onto Makefile changes for v2.1.0-rc0]
[AF: Rebased onto script rewrite for v2.7.0-rc2 - to be fixed]
Signed-off-by: Andreas Färber <afaerber@suse.de>
---
 linux-user/binfmt.c | 42 ++++++++++++++++++++++++++++++++++++++++++
 meson.build         |  5 +++++
 2 files changed, 47 insertions(+)

Index: qemu-7.1.0/linux-user/binfmt.c
===================================================================
--- /dev/null
+++ qemu-7.1.0/linux-user/binfmt.c
@@ -0,0 +1,42 @@
+#include <stdio.h>
+#include <stdarg.h>
+#include <unistd.h>
+#include <libgen.h>
+#include <string.h>
+#include <stdlib.h>
+
+
+int main(int argc, char **argv, char **envp)
+{
+    char *binfmt;
+    char **new_argv;
+
+    /*
+     * Check if our file name ends with -binfmt
+     */
+    binfmt = argv[0] + strlen(argv[0]) - strlen("-binfmt");
+    if (strcmp(binfmt, "-binfmt")) {
+        fprintf(stderr, "%s: Invalid executable name\n", argv[0]);
+        exit(1);
+    }
+    if (argc < 3) {
+        fprintf(stderr, "%s: Please use me through binfmt with P flag\n",
+                argv[0]);
+        exit(1);
+    }
+
+    binfmt[0] = '\0';
+    /* Now argv[0] is the real qemu binary name */
+
+    new_argv = (char **)malloc((argc + 2) * sizeof(*new_argv));
+    if (argc > 3) {
+        memcpy(&new_argv[4], &argv[3], (argc - 3) * sizeof(*new_argv));
+    }
+    new_argv[0] = argv[0];
+    new_argv[1] = (char *)"-0";
+    new_argv[2] = argv[2];
+    new_argv[3] = argv[1];
+    new_argv[argc + 1] = NULL;
+
+    return execve(new_argv[0], new_argv, envp);
+}
Index: qemu-7.1.0/meson.build
===================================================================
--- qemu-7.1.0.orig/meson.build
+++ qemu-7.1.0/meson.build
@@ -3585,6 +3585,11 @@ endforeach
 
 # Other build targets
 
+if 'CONFIG_LINUX_USER' in config_target
+    executable('qemu-binfmt', files('linux-user/binfmt.c'),
+               install: true)
+endif
+
 if 'CONFIG_PLUGIN' in config_host
   install_headers('include/qemu/qemu-plugin.h')
 endif
Index: qemu-7.1.0/scripts/qemu-binfmt-conf.sh
===================================================================
--- qemu-7.1.0.orig/scripts/qemu-binfmt-conf.sh
+++ qemu-7.1.0/scripts/qemu-binfmt-conf.sh
@@ -272,6 +272,7 @@ qemu_check_systemd() {
 
 qemu_generate_register() {
     flags=""
+    binfmt=
     if [ "$CREDENTIAL" = "yes" ] ; then
         flags="OC"
     fi
@@ -280,9 +281,10 @@ qemu_generate_register() {
     fi
     if [ "$PRESERVE_ARG0" = "yes" ] ; then
         flags="${flags}P"
+	binfmt=-binfmt
     fi
 
-    echo ":qemu-$cpu:M::$magic:$mask:$qemu:$flags"
+    echo ":qemu-$cpu$binfmt:M::$magic:$mask:$qemu:$flags"
 }
 
 qemu_register_interpreter() {

Places

File linux-user-add-binfmt-wrapper-for-argv-0.patch of Package qemu (Revision 43909338b979fda35565a21db433bd85)

Places