Re: [UFS] Broken suiddir? (+patch)

From: Pawel Jakub Dawidek <pjd_at_FreeBSD.org>
Date: Wed, 24 Mar 2004 23:55:44 +0100
On Wed, Mar 24, 2004 at 11:53:18PM +0100, Pawel Jakub Dawidek wrote:
+> On Tue, Mar 23, 2004 at 11:30:20PM +0100, Rafal Skoczylas wrote:
+> +> root_at_hawaje:/usr/src/sys/ufs/ufs% diff -u ufs_vnops.c.orig ufs_vnops.c
+> +> --- ufs_vnops.c.orig    Sat Oct 18 16:10:27 2003
+> +> +++ ufs_vnops.c Sun Mar 21 01:31:04 2004
+> +> _at__at_ -1411,7 +1411,7 _at__at_
+> +>                                  * our context so a stack variable is ok.
+> +>                                  */
+> +>                                 ucred.cr_ref = 1;
+> +> -                               ucred.cr_uid = ip->i_uid;
+> +> +                               ucred.cr_uid = dp->i_uid;
+> +>                                 ucred.cr_ngroups = 1;
+> +>                                 ucred.cr_groups[0] = dp->i_gid;
+> +>                                 ucp = &ucred;
+> +> root_at_hawaje:/usr/src/sys/ufs/ufs%
+> 
+> I haven't test your report and patch yet, but it is strange.
+> In line 1403 stays:
+> 
+> 	ip->i_uid = dp->i_uid;
+> 
+> So ip->i_uid should be equal in line which you have patched.

s/should be equal/should be equal to dp->i_uid/

-- 
Pawel Jakub Dawidek                       http://www.FreeBSD.org
pjd_at_FreeBSD.org                           http://garage.freebsd.pl
FreeBSD committer                         Am I Evil? Yes, I Am!

Received on Wed Mar 24 2004 - 13:55:46 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:48 UTC