Date: Wed, 24 Mar 2004 23:55:44 +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: <20040324225544.GT8930@darkness.comp.waw.pl> In-Reply-To: <20040324225318.GS8930@darkness.comp.waw.pl> References: <20040323223020.GA2931@secprog.org> <20040324225318.GS8930@darkness.comp.waw.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
--bNOIqPvWVsuhXMpy Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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@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% +>=20 +> I haven't test your report and patch yet, but it is strange. +> In line 1403 stays: +>=20 +> ip->i_uid =3D dp->i_uid; +>=20 +> So ip->i_uid should be equal in line which you have patched. s/should be equal/should be equal to dp->i_uid/ --=20 Pawel Jakub Dawidek http://www.FreeBSD.org pjd@FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am! --bNOIqPvWVsuhXMpy Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (FreeBSD) iD8DBQFAYhHwForvXbEpPzQRAvGUAKCS693hO79fiutmvp0Pzcc/dLosPwCghHQB MV8k4RTcEK6VZUdkTY/bxoA= =9nt2 -----END PGP SIGNATURE----- --bNOIqPvWVsuhXMpy--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040324225544.GT8930>