Skip to content
Permalink
Browse files

6388 Failure of userland copy should return EFAULT

Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: Dan Kimmel <dan.kimmel@delphix.com>
Approved by: Robert Mustacchi <rm@joyent.com>
  • Loading branch information...
ryao authored and ahrens committed Oct 31, 2015
1 parent 7c5a457 commit c71c00bbe8a9cdc7e3f4048b751f48e80441d506
Showing with 1 addition and 1 deletion.
  1. +1 −1 usr/src/uts/common/fs/zfs/zfs_ioctl.c
@@ -1303,7 +1303,7 @@ get_nvlist(uint64_t nvl, uint64_t size, int iflag, nvlist_t **nvp)
if ((error = ddi_copyin((void *)(uintptr_t)nvl, packed, size,
iflag)) != 0) {
kmem_free(packed, size);
return (error);
return (SET_ERROR(EFAULT));
}

if ((error = nvlist_unpack(packed, size, &list, 0)) != 0) {

0 comments on commit c71c00b

Please sign in to comment.
You can’t perform that action at this time.