From: Ben Hutchings <bhutchings@solarflare.com>
To: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Cc: <eric.dumazet@gmail.com>, <shemminger@vyatta.com>,
<netdev@vger.kernel.org>, <junwei.zhang@6wind.com>
Subject: Re: [PATCH iproute2 v4] ipbatch: fix use of 'ip netns exec'
Date: Tue, 9 Jul 2013 17:19:24 +0100 [thread overview]
Message-ID: <1373386764.2523.1.camel@bwh-desktop.uk.level5networks.com> (raw)
In-Reply-To: <1373385312-4899-1-git-send-email-nicolas.dichtel@6wind.com>
On Tue, 2013-07-09 at 17:55 +0200, Nicolas Dichtel wrote:
[...]
> diff --git a/ip/ipnetns.c b/ip/ipnetns.c
> index fa2b681..cdc3101 100644
> --- a/ip/ipnetns.c
> +++ b/ip/ipnetns.c
> @@ -42,6 +42,7 @@
> #define MS_SHARED (1 << 20)
> #endif
>
> +extern char *batch_file;
Should be declared in a header file.
> #ifndef HAVE_SETNS
> static int setns(int fd, int nstype)
> @@ -185,6 +186,42 @@ static int netns_exec(int argc, char **argv)
> /* Setup bind mounts for config files in /etc */
> bind_etc(name);
>
> + if (batch_file) {
> + int status;
> + pid_t pid;
> +
> + pid = fork();
> + if (pid < 0) {
> + perror("fork");
> + return EXIT_FAILURE;
return -1;
> + }
> +
> + if (pid == 0) {
> + /* Child */
> + if (execvp(cmd, argv + 1) < 0)
> + fprintf(stderr, "exec of \"%s\" failed: %s\n",
> + cmd, strerror(errno));
> +
> + _exit(EXIT_FAILURE);
> + }
> +
> + /* Parent */
> + if (waitpid(pid, &status, 0) < 0) {
> + perror("waitpid");
> + return EXIT_FAILURE;
return -1;
> + }
> +
> + if (WIFEXITED(status)) {
> + /* ip must returns the status of the child, but do_cmd()
> + * will add a minus to this returned value, so let's add
> + * another one here to cancel it.
> + */
> + return -WEXITSTATUS(status);
> + }
> +
> + return EXIT_FAILURE;
return -1;
> + }
> +
> if (execvp(cmd, argv + 1) < 0)
> fprintf(stderr, "exec of \"%s\" failed: %s\n",
> cmd, strerror(errno));
--
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
next prev parent reply other threads:[~2013-07-09 16:19 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20130705141639.56b47e7e@samsung-9>
2013-07-08 9:44 ` [PATCH iproute2 v2] ipbatch: fix use of 'ip netns exec' Nicolas Dichtel
2013-07-08 23:36 ` Ben Hutchings
2013-07-09 10:26 ` [PATCH iproute2 v3] " Nicolas Dichtel
2013-07-09 13:20 ` Eric Dumazet
2013-07-09 15:55 ` [PATCH iproute2 v4] " Nicolas Dichtel
2013-07-09 16:19 ` Ben Hutchings [this message]
2013-07-09 16:59 ` Nicolas Dichtel
2013-07-09 18:13 ` Ben Hutchings
2013-07-10 9:52 ` Nicolas Dichtel
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=1373386764.2523.1.camel@bwh-desktop.uk.level5networks.com \
--to=bhutchings@solarflare.com \
--cc=eric.dumazet@gmail.com \
--cc=junwei.zhang@6wind.com \
--cc=netdev@vger.kernel.org \
--cc=nicolas.dichtel@6wind.com \
--cc=shemminger@vyatta.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).