Date: Fri, 19 Apr 2019 11:34:10 -0700 From: Enji Cooper <yaneurabeya@gmail.com> To: Alexander Motin <mav@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r346390 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs Message-ID: <BAB69FEF-6F73-467C-8569-801E6CA96198@gmail.com> In-Reply-To: <8d763a63-0915-d9f6-1b90-5e902f06ebe4@FreeBSD.org> References: <201904191544.x3JFijLe075969@repo.freebsd.org> <0F708A9C-D29D-466A-B541-06DB7127349E@gmail.com> <8d763a63-0915-d9f6-1b90-5e902f06ebe4@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Apr 19, 2019, at 11:29, Alexander Motin <mav@freebsd.org> wrote: >=20 > On 19.04.2019 14:09, Enji Cooper wrote: >>> On Apr 19, 2019, at 08:44, Alexander Motin <mav@freebsd.org> wrote: >>>=20 >>> Author: mav >>> Date: Fri Apr 19 15:44:45 2019 >>> New Revision: 346390 >>> URL: https://svnweb.freebsd.org/changeset/base/346390 >>>=20 >>> Log: >>> Change the way FreeBSD GID inheritance is hacked. >>>=20 >>> I believe previous ifdef caused NULL dereference in later zfs_log_create= () >>> on attempt to create file inside directory belonging to ephemeral group >>> created on illumos, trying to write to log information about GID domain >>> of the newly created file, inheriting the ephemeral GID. >>>=20 >>> This patch reuses original illumos SGID code with exception that due to >>> lack of ID mapping code on FreeBSD ephemeral GID will turn into GID_NOBO= DY >>> by another ifdef inside zfs_fuid_map_id(). >>=20 >> Hi mav@! >> I was wondering, does this break ACL mappings with sticky bits for the= group ID (or was it broken to begin with)? If the latter, does it make sens= e to file a bug/add a TODO comment? >=20 > I don't believe this change should have any visible results for native > FreeBSD pools. It does change change handling of some pools coming from > illumos, but previous behavior in that case resulted in kernel panic, so > anything else is better, while full compatibility is impossible, since > FreeBSD has no kernel-side AD UID/GID mapping, used by illumos SMB. >=20 > Could you better describe what ACL mapping breakage you are talking about?= Ah... Yeah, it makes sense to leave this alone since it=E2=80=99s a miss= ing ZFS on FreeBSD feature. Thank you! -Enji PS I was likely messing up my interpretation of the stat field. I didn=E2=80= =99t read the manpage yet.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?BAB69FEF-6F73-467C-8569-801E6CA96198>