svn commit: r195909 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys

Pawel Jakub Dawidek pjd at FreeBSD.org
Mon Jul 27 14:52:35 UTC 2009


Author: pjd
Date: Mon Jul 27 14:52:34 2009
New Revision: 195909
URL: http://svn.freebsd.org/changeset/base/195909

Log:
  We don't support ephemeral IDs in FreeBSD and without this fix ZFS can
  panic when in zfs_fuid_create_cred() when userid is negative. It is
  converted to unsigned value which makes IS_EPHEMERAL() macro to
  incorrectly report that this is ephemeral ID. The most reasonable
  solution for now is to always report that the given ID is not ephemeral.
  
  PR:		kern/132337
  Submitted by:	Matthew West <freebsd at r.zeeb.org>
  Tested by:	Thomas Backman <serenity at exscape.org>, Michael Reifenberger <mike at reifenberger.com>
  Approved by:	re (kib)
  MFC after:	2 weeks

Modified:
  head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_znode.h

Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_znode.h
==============================================================================
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_znode.h	Mon Jul 27 14:22:09 2009	(r195908)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_znode.h	Mon Jul 27 14:52:34 2009	(r195909)
@@ -78,7 +78,11 @@ extern "C" {
 /*
  * Is ID ephemeral?
  */
+#ifdef TODO
 #define	IS_EPHEMERAL(x)		(x > MAXUID)
+#else
+#define	IS_EPHEMERAL(x)		(0)
+#endif
 
 /*
  * Should we use FUIDs?


More information about the svn-src-all mailing list