diff options
author | Richard Yao <[email protected]> | 2015-07-03 12:20:17 -0400 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2015-07-14 10:20:35 -0700 |
commit | 0de7c552b69f2bb3e16f023007f049cb3c8e9afd (patch) | |
tree | 08beaa1d0901b761ce3ed2d1e5e44413f22a4d46 /cmd/zpool/zpool_main.c | |
parent | b39c22b73c0e8016381057c2240570f7af992def (diff) |
Failure of userland copy should return EFAULT
Many key internal functions pass system return codes that are safe to
return to userland. In the case of ddi_copyin(9F), an error passes -1
and the documentation states very clearly that drivers should pass
EFAULT to userland when this happens.
http://illumos.org/man/9F/ddi_copyin
This does not happen in the ZFS source code. I believe it should be
changed to pass EFAULT. I caught this when writing man pages for the
libzfs_core API.
Signed-off-by: Richard Yao <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #3575
Diffstat (limited to 'cmd/zpool/zpool_main.c')
0 files changed, 0 insertions, 0 deletions