svn commit: r322218 - head/cddl/contrib/opensolaris/lib/libzfs/common

Andriy Gapon avg at FreeBSD.org
Tue Aug 8 10:30:50 UTC 2017


Author: avg
Date: Tue Aug  8 10:30:49 2017
New Revision: 322218
URL: https://svnweb.freebsd.org/changeset/base/322218

Log:
  MFV r322217: 8418 zfs_prop_get_table() call in zfs_validate_name() is a no-op
  
  illumos/illumos-gate at e09ba01dcda5e24964b8632718777b39166d86e4
  https://github.com/illumos/illumos-gate/commit/e09ba01dcda5e24964b8632718777b39166d86e4
  
  https://www.illumos.org/issues/8418
    The following line in zfs_validate_name() is just a no-op and it
    should be removed:
        108    (void) zfs_prop_get_table();
  
  Reviewed by: Vitaliy Gusev <gusev.vitaliy at icloud.com>
  Approved by: Matthew Ahrens <mahrens at delphix.com>
  Author: Marcel Telka <marcel at telka.sk>
  
  MFC after:	2 weeks

Modified:
  head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_dataset.c
Directory Properties:
  head/cddl/contrib/opensolaris/   (props changed)
  head/cddl/contrib/opensolaris/lib/libzfs/   (props changed)

Modified: head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_dataset.c
==============================================================================
--- head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_dataset.c	Tue Aug  8 10:28:01 2017	(r322217)
+++ head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_dataset.c	Tue Aug  8 10:30:49 2017	(r322218)
@@ -103,7 +103,6 @@ zfs_validate_name(libzfs_handle_t *hdl, const char *pa
 	namecheck_err_t why;
 	char what;
 
-	(void) zfs_prop_get_table();
 	if (entity_namecheck(path, &why, &what) != 0) {
 		if (hdl != NULL) {
 			switch (why) {


More information about the svn-src-all mailing list