git: sbin/hammer: Fix/remove redundant error variable
Tomohiro Kusumi
tkusumi at crater.dragonflybsd.org
Thu Dec 15 11:21:45 PST 2016
commit e671cfa600999a44074bfd4f62c3346b02178217
Author: Tomohiro Kusumi <kusumi.tomohiro at gmail.com>
Date: Thu Dec 15 18:16:36 2016 +0900
sbin/hammer: Fix/remove redundant error variable
Check ioctl result right after ioctl.
Since this for-loop expects ioctl(GET_PSEUDOFS) to eventually
return ENOENT for a new slot, error variable is likely to be
overwritten with ENOENT by the time for-loop ends. This could
result in overlooking possible real errors.
If errno is checked (if not ENOENT) right after ioctl, there's
also no need to preserve errno value.
Summary of changes:
sbin/hammer/cmd_pfs.c | 14 +++++---------
1 file changed, 5 insertions(+), 9 deletions(-)
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/e671cfa600999a44074bfd4f62c3346b02178217
--
DragonFly BSD source repository
More information about the Commits
mailing list