[prev in list] [next in list] [prev in thread] [next in thread] 

List:       strace
Subject:    Re: [PATCH] Implement decoding of NS_* ioctl commands
From:       "Dmitry V. Levin" <ldv () altlinux ! org>
Date:       2017-04-14 2:49:28
Message-ID: 20170414024928.GE14070 () altlinux ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Thu, Apr 13, 2017 at 10:08:36PM +0300, Dmitry V. Levin wrote:
> On Thu, Apr 13, 2017 at 06:58:00PM +0300, Dmitry V. Levin wrote:
> > On Thu, Apr 13, 2017 at 10:10:11PM +0700, Nikolay Marchuk wrote:
[...]
> > > +	pid = clone(child, tail_alloc(1) + 1,
> > > +	            CLONE_NEWUSER | CLONE_UNTRACED | SIGCHLD, pipefd);
[...]
> OK, I've applied these trivial changes myself and pushed it.
> Thanks!

And two more commits on top of it, to workaround issues with this clone()
call on niche architectures.

First, there is no clone() on ia64, but there is a __clone2() with a
slightly different semantics instead.  The latter is documented but
no prototype is provided by glibc.

Second, it crashes with SIGBUS on hppa which is not happy with the
child_stack address.

When using obscure functions like clone(), issues are popping up
like mushrooms after the rain.


-- 
ldv

["signature.asc" (application/pgp-signature)]

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


[prev in list] [next in list] [prev in thread] [next in thread]