<p dir="ltr">Hi,<br></p>
<p dir="ltr">Le 23 nov. 2015 08:18, "John Crispin" <<a href="mailto:blogic@openwrt.org">blogic@openwrt.org</a>> a écrit :<br>
><br>
><br>
><br>
> On 23/11/2015 01:39, Etienne CHAMPETIER wrote:<br>
> > spawn_jail(void) produce a compilation error,<br>
> > so we use spawn_jail()<br>
> ><br>
> > Signed-off-by: Etienne CHAMPETIER <<a href="mailto:champetier.etienne@gmail.com">champetier.etienne@gmail.com</a>><br>
> > ---<br>
> >  jail/jail.c | 4 ++--<br>
> >  1 file changed, 2 insertions(+), 2 deletions(-)<br>
> ><br>
> > diff --git a/jail/jail.c b/jail/jail.c<br>
> > index 56dc9ca..08babde 100644<br>
> > --- a/jail/jail.c<br>
> > +++ b/jail/jail.c<br>
> > @@ -272,7 +272,7 @@ static int exec_jail()<br>
> >       exit(EXIT_FAILURE);<br>
> >  }<br>
> ><br>
> > -static int spawn_jail(void *arg)<br>
> > +static int spawn_jail()<br>
><br>
> Hi,<br>
><br>
> this is still wrong. also you might want to write a sentence why it is<br>
> useless.<br>
></p>
<p dir="ltr">Well, we don't use it, and passing arg to not use it really doesn't improve readability.</p>
<p dir="ltr">I know it break your code style, but it seems we can't have both here :(</p>
<p dir="ltr">Etienne</p>
<p dir="ltr">>         John<br>
><br>
> >  {<br>
> >       if (<a href="http://opts.name">opts.name</a> && sethostname(<a href="http://opts.name">opts.name</a>, strlen(<a href="http://opts.name">opts.name</a>))) {<br>
> >               ERROR("failed to sethostname: %s\n", strerror(errno));<br>
> > @@ -424,7 +424,7 @@ int main(int argc, char **argv)<br>
> >       if (opts.namespace) {<br>
> >               jail_process.pid = clone(spawn_jail,<br>
> >                       child_stack + STACK_SIZE,<br>
> > -                     CLONE_NEWUTS | CLONE_NEWPID | CLONE_NEWNS | CLONE_NEWIPC | SIGCHLD, argv);<br>
> > +                     CLONE_NEWUTS | CLONE_NEWPID | CLONE_NEWNS | CLONE_NEWIPC | SIGCHLD, NULL);<br>
> >       } else {<br>
> >               jail_process.pid = fork();<br>
> >       }<br>
> ><br>
</p>