Re: [patch] tame.2 documentation about systrace.4

2015-09-20 Thread Jason McIntyre
On Sun, Sep 20, 2015 at 02:57:47PM +0200, Sebastian Benoit wrote:
> Sebastien Marie(sema...@openbsd.org) on 2015.09.20 14:27:01 +0200:
> > Hi,
> > 
> > Mentions that using systrace(4) isn't possible when a program has called
> > tame(2).
> > 
> > Comments ? OK ?
> > -- 
> > Sebastien Marie
> > 
> > Index: lib/libc/sys/tame.2
> > ===
> > RCS file: /cvs/src/lib/libc/sys/tame.2,v
> > retrieving revision 1.27
> > diff -u -p -r1.27 tame.2
> > --- lib/libc/sys/tame.2 11 Sep 2015 09:01:16 -  1.27
> > +++ lib/libc/sys/tame.2 20 Sep 2015 12:26:47 -
> > @@ -431,3 +431,7 @@ The
> >  .Fn tame
> >  system call appeared in
> >  .Ox 5.8 .
> > +.Sh CAVEATS
> > +The use of
> > +.Xr systrace 4
> > +in a tamed program is disable.
> 
> there is a "d" missing at the end.
> 
> And "In a tamed program, systrace(4) is disabled" sounds better to me, but
> jmc might have an opinion on that.
> 
> This could also be mentioned in the systrace(4) manpage, after all, if your
> wondering why systrace doesnt work, you won't know that the program is
> tamed and thus won't look in tame(2)?
> 

i wouldn;t worry about the exact wording too much, but it probably does
make sense to flag it in systrace(4) too.

jmc



Re: [patch] tame.2 documentation about systrace.4

2015-09-20 Thread Jona Joachim
On 2015-09-20, Sebastien Marie  wrote:
> Hi,
>
> Mentions that using systrace(4) isn't possible when a program has called
> tame(2).
>
> Comments ? OK ?

I think you mean "is disable*d*"



[patch] tame.2 documentation about systrace.4

2015-09-20 Thread Sebastien Marie
Hi,

Mentions that using systrace(4) isn't possible when a program has called
tame(2).

Comments ? OK ?
-- 
Sebastien Marie

Index: lib/libc/sys/tame.2
===
RCS file: /cvs/src/lib/libc/sys/tame.2,v
retrieving revision 1.27
diff -u -p -r1.27 tame.2
--- lib/libc/sys/tame.2 11 Sep 2015 09:01:16 -  1.27
+++ lib/libc/sys/tame.2 20 Sep 2015 12:26:47 -
@@ -431,3 +431,7 @@ The
 .Fn tame
 system call appeared in
 .Ox 5.8 .
+.Sh CAVEATS
+The use of
+.Xr systrace 4
+in a tamed program is disable.



Re: [patch] tame.2 documentation about systrace.4

2015-09-20 Thread Sebastian Benoit
Sebastien Marie(sema...@openbsd.org) on 2015.09.20 14:27:01 +0200:
> Hi,
> 
> Mentions that using systrace(4) isn't possible when a program has called
> tame(2).
> 
> Comments ? OK ?
> -- 
> Sebastien Marie
> 
> Index: lib/libc/sys/tame.2
> ===
> RCS file: /cvs/src/lib/libc/sys/tame.2,v
> retrieving revision 1.27
> diff -u -p -r1.27 tame.2
> --- lib/libc/sys/tame.2   11 Sep 2015 09:01:16 -  1.27
> +++ lib/libc/sys/tame.2   20 Sep 2015 12:26:47 -
> @@ -431,3 +431,7 @@ The
>  .Fn tame
>  system call appeared in
>  .Ox 5.8 .
> +.Sh CAVEATS
> +The use of
> +.Xr systrace 4
> +in a tamed program is disable.

there is a "d" missing at the end.

And "In a tamed program, systrace(4) is disabled" sounds better to me, but
jmc might have an opinion on that.

This could also be mentioned in the systrace(4) manpage, after all, if your
wondering why systrace doesnt work, you won't know that the program is
tamed and thus won't look in tame(2)?