jail getfsstat patches.

Pawel Jakub Dawidek pjd at FreeBSD.org
Sun Jun 27 10:40:44 PDT 2004


On Sun, Jun 27, 2004 at 05:29:23PM +0000, Bjoern A. Zeeb wrote:
+> some more points while skipping through the code a second time:
+> 
+> * could we make this
+> 	strlcpy(sp->f_mntonname, "<restricted>",
+>   s.th. w/o '<' and '>' ? [] or () would make me feel better as
+>   those though special characters won't redirect anything in sh
+>   even if we should never see this.

Ok, will do.

+> * in fhstatfs you missed a prison_enforce_statfs call

Nope. It is placed in cvtstatfs().

+> * some spelling/grammer left to someone else

Yeah.

+> else looks good to me; didn't check the compat code;
+> going to see if I can build a kernel and setup some jails on a
+> test machine to see if it works as well.

Thanks!

-- 
Pawel Jakub Dawidek                       http://www.FreeBSD.org
pjd at FreeBSD.org                           http://garage.freebsd.pl
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20040627/4a0d8513/attachment.bin


More information about the freebsd-current mailing list