Author: mav
Date: Wed Aug 28 17:39:46 2019
New Revision: 351574
URL: https://svnweb.freebsd.org/changeset/base/351574

Log:
  MFV/ZoL: Fix wrong assertion in libzfs diff error handling
  
  In compare(), all error cases set the error code to EPIPE, so when an
  error is set, the correct assertion to make is that the error is EPIPE,
  not EINVAL.
  
  Reviewed-by: Richard Elling <[email protected]>
  Reviewed-by: Brian Behlendorf <[email protected]>
  Signed-off-by: Ryan Moeller <[email protected]>
  Closes #8743
  zfsonlinux/zfs@9dc41a769df164875d974c2431b2453e70e16c41
  
  Submitted by: Ryan Moeller <[email protected]>
  MFC after:    1 week
  Sponsored by: iXsystems, Inc.
  Differential Revision:        https://reviews.freebsd.org/D20118

Modified:
  head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_diff.c

Modified: head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_diff.c
==============================================================================
--- head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_diff.c       Wed Aug 
28 17:01:28 2019        (r351573)
+++ head/cddl/contrib/opensolaris/lib/libzfs/common/libzfs_diff.c       Wed Aug 
28 17:39:46 2019        (r351574)
@@ -472,7 +472,7 @@ differ(void *arg)
        if (err)
                return ((void *)-1);
        if (di->zerr) {
-               ASSERT(di->zerr == EINVAL);
+               ASSERT(di->zerr == EPIPE);
                (void) snprintf(di->errbuf, sizeof (di->errbuf),
                    dgettext(TEXT_DOMAIN,
                    "Internal error: bad data from diff IOCTL"));
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to