On Sun, Jun 27, 2004 at 06:32:49PM +0000, Bjoern A. Zeeb wrote: +> > It starts from: +> > +> > error = suser(td); +> > if (error) +> > return (error); +> > +> > So I don't think it is needed here, but we can put this there just in +> > case of future changes. +> +> for that and perhaps just for consistency with freebsd4_fhstatfs ? This is not intended in freebsd4_fhstatfs(). It is there only because cvtstatfs() is used also by freebsd4_statfs(), freebsd4_fstatfs() and freebsd4_getfsstat(). Anyway. I updated the patch. -- Pawel Jakub Dawidek http://www.FreeBSD.org pjd_at_FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am!
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:59 UTC