Date: Wed, 24 Mar 2004 23:53:18 +0100 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Rafal Skoczylas <nils@secprog.org> Cc: freebsd-current@freebsd.org Subject: Re: [UFS] Broken suiddir? (+patch) Message-ID: <20040324225318.GS8930@darkness.comp.waw.pl> In-Reply-To: <20040323223020.GA2931@secprog.org> References: <20040323223020.GA2931@secprog.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--I2AcQh+/kfs26T/w Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Mar 23, 2004 at 11:30:20PM +0100, Rafal Skoczylas wrote: +> root@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 +> @@ -1411,7 +1411,7 @@ +> * our context so a stack variable is ok. +> */ +> ucred.cr_ref =3D 1; +> - ucred.cr_uid =3D ip->i_uid; +> + ucred.cr_uid =3D dp->i_uid; +> ucred.cr_ngroups =3D 1; +> ucred.cr_groups[0] =3D dp->i_gid; +> ucp =3D &ucred; +> root@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 =3D dp->i_uid; So ip->i_uid should be equal in line which you have patched. --=20 Pawel Jakub Dawidek http://www.FreeBSD.org pjd@FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am! --I2AcQh+/kfs26T/w Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (FreeBSD) iD8DBQFAYhFeForvXbEpPzQRAjDCAKCCSbN5bguG9TN3TYz8txyjoARc4ACgiMfK 3kDKfpoAT6e2+f3beZVBZbY= =GCq/ -----END PGP SIGNATURE----- --I2AcQh+/kfs26T/w--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040324225318.GS8930>