Date: Fri, 26 Aug 2005 12:41:49 +0200 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Takanori Watanabe <takawata@init-main.com> Cc: takawata@freebsd.org, freebsd-geom@freebsd.org Subject: Re: NTFS label patch. Message-ID: <20050826104149.GD43700@garage.freebsd.pl> In-Reply-To: <200508261017.j7QAHgTo069421@ns.init-main.com> References: <200508261017.j7QAHgTo069421@ns.init-main.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--7DO5AaGCk89r4vaK Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Aug 26, 2005 at 07:17:42PM +0900, Takanori Watanabe wrote: +> Hi, I wrote NTFS volume label parser for geom_label. +> Now I mount NTFS by volume label-based name. +> If you interested in, plaese check. +> Please Cc: to me=20 +> Thanks. +>=20 +> http://www.init-main.com/gntfs/ Looks ok, but it needs one change for sure. When you do it: filerecp =3D g_read_data(cp, bf->bf_mftcn * bf->bf_spc * bf->bf_bps + recsi= ze * NTFS_VOLUMEINO, recsize, &error); you need to be sure that offset is ok: off_t offset; offset =3D bf->bf_mftcn * bf->bf_spc * bf->bf_bps + recsize * NTFS_VOLUMEIN= O; if ((offset % pp->sectorsize) !=3D 0) goto done; filerecp =3D g_read_data(cp, offset, recsize, &error); With this change it should be ok from GEOM/g_label point of view, so fell free to commit. Don't forget to update manual page. Thanks. --=20 Pawel Jakub Dawidek http://www.wheel.pl pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! --7DO5AaGCk89r4vaK Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.1 (FreeBSD) iD8DBQFDDvHtForvXbEpPzQRAoPeAJwLpSQIOlEULN69z5o0RigobYRg7gCfaNWN eY/SSxRyivFprjuSJsNXWGE= =5MgW -----END PGP SIGNATURE----- --7DO5AaGCk89r4vaK--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050826104149.GD43700>