All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Christian de Rivaz <jc@eclis.ch>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH] script=no for the TUN/TAP net option
Date: Wed, 31 Jan 2007 15:42:07 +0100	[thread overview]
Message-ID: <45C0AABF.6030501@eclis.ch> (raw)

[-- Attachment #1: Type: text/plain, Size: 480 bytes --]

Hello,

The following small patch disable the execution of a script while using 
the -net tap option with a already existing TUN/TAP interface.

So you can as root bring up a TUN/TAP interface to a user with the 
command 'tunctl -u <user> -t <ifname>' and setup the routing. Then the 
user don't need to sudo a script, it simply tell QEMU to use the 
interface <ifname> with the option '-net nic -net 
tap,ifname=<ifname>,script=no'.

Have a good day,
-- 
Jean-Christian de Rivaz

[-- Attachment #2: net-tap-script-no.patch.txt --]
[-- Type: text/plain, Size: 1835 bytes --]

Index: qemu-doc.texi
===================================================================
RCS file: /sources/qemu/qemu/qemu-doc.texi,v
retrieving revision 1.125
diff -u -r1.125 qemu-doc.texi
--- qemu-doc.texi	28 Jan 2007 01:53:16 -0000	1.125
+++ qemu-doc.texi	31 Jan 2007 14:29:48 -0000
@@ -358,7 +358,8 @@
 @item -net tap[,vlan=n][,fd=h][,ifname=name][,script=file]
 Connect the host TAP network interface @var{name} to VLAN @var{n} and
 use the network script @var{file} to configure it. The default
-network script is @file{/etc/qemu-ifup}. If @var{name} is not
+network script is @file{/etc/qemu-ifup}. Use @option{script=no} to
+disable script execution. If @var{name} is not
 provided, the OS automatically provides one.  @option{fd=h} can be
 used to specify the handle of an already opened host TAP interface. Example:
 
Index: vl.c
===================================================================
RCS file: /sources/qemu/qemu/vl.c,v
retrieving revision 1.244
diff -u -r1.244 vl.c
--- vl.c	28 Jan 2007 01:53:16 -0000	1.244
+++ vl.c	31 Jan 2007 14:29:49 -0000
@@ -3289,7 +3289,7 @@
     if (fd < 0)
         return -1;
 
-    if (!setup_script)
+    if (!setup_script || !strcmp(setup_script, "no"))
         setup_script = "";
     if (setup_script[0] != '\0') {
         /* try to launch network init script */
@@ -6066,6 +6066,7 @@
            "-net tap[,vlan=n][,fd=h][,ifname=name][,script=file]\n"
            "                connect the host TAP network interface to VLAN 'n' and use\n"
            "                the network script 'file' (default=%s);\n"
+           "                use 'script=no' to disable script execution;\n"
            "                use 'fd=h' to connect to an already opened TAP interface\n"
 #endif
            "-net socket[,vlan=n][,fd=h][,listen=[host]:port][,connect=host:port]\n"

                 reply	other threads:[~2007-01-31 14:42 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45C0AABF.6030501@eclis.ch \
    --to=jc@eclis.ch \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.