On Tue, Jul 31, 2018 at 06:39:18PM -0500, Andrew Daugherity wrote:
[...] 
> Related: in the same section of code (at the end of my diff actually,
> as context), I noticed that when -u is used, the chroot path is set to
> the target user's home directory instead of /var/www.  I found this
> surprising, so I added a manpage diff to my patchset:
> ====
> --- slowcgi.8 2017-10-17 17:47:58.000000000 -0500
> +++ slowcgi.8 2018-07-26 13:34:06.459779115 -0500
> @@ -78,7 +78,9 @@
>  .It Fl u Ar user
>  Drop privileges to
>  .Ar user
> -instead of default user www.
> +instead of the default www, and chroot to that user's home directory,
> +unless you specify otherwise with
> +.Ar -p .
>  .El
>  .Sh SEE ALSO
>  .Xr httpd 8
> ====
> Perhaps that's a bit too wordy and only the first line is needed, I dunno.

How about this? jmc?

diff --git slowcgi.8 slowcgi.8
index 52bded7eee6..117228403b4 100644
--- slowcgi.8
+++ slowcgi.8
@@ -79,6 +79,12 @@ Create and bind to alternative local socket at
 Drop privileges to
 .Ar user
 instead of default user www.
+.Nm
+will
+.Xr chroot 8
+to
+the home directory of
+.Ar user .
 .El
 .Sh SEE ALSO
 .Xr httpd 8

Jmc: Btw, why are we using the section 8 man page in Xr, technically
slowcgi uses chroot(2).

-- 
I'm not entirely sure you are real.

Reply via email to