diff options
| author | yamt <yamt@NetBSD.org> | 2006-01-08 09:09:53 +0000 |
|---|---|---|
| committer | yamt <yamt@NetBSD.org> | 2006-01-08 09:09:53 +0000 |
| commit | 5bca923bc44fbe271acc5df454dbccad74f72837 (patch) | |
| tree | 359b61509568d6356aaae1a3c98bfbd22a516fcd /sys/dev/raidframe | |
| parent | fe3aca751fd5ff478b0147af82e0aec36891a8e1 (diff) | |
raidioctl:
- don't ignore raidread_component_label error.
- remove an unnessesary memset.
Diffstat (limited to 'sys/dev/raidframe')
| -rw-r--r-- | sys/dev/raidframe/rf_netbsdkintf.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/sys/dev/raidframe/rf_netbsdkintf.c b/sys/dev/raidframe/rf_netbsdkintf.c index 97b1e11bcca..0da9f0ea832 100644 --- a/sys/dev/raidframe/rf_netbsdkintf.c +++ b/sys/dev/raidframe/rf_netbsdkintf.c @@ -1,4 +1,4 @@ -/* $NetBSD: rf_netbsdkintf.c,v 1.194 2006/01/07 16:08:44 oster Exp $ */ +/* $NetBSD: rf_netbsdkintf.c,v 1.195 2006/01/08 09:09:53 yamt Exp $ */ /*- * Copyright (c) 1996, 1997, 1998 The NetBSD Foundation, Inc. * All rights reserved. @@ -146,7 +146,7 @@ ***********************************************************/ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: rf_netbsdkintf.c,v 1.194 2006/01/07 16:08:44 oster Exp $"); +__KERNEL_RCSID(0, "$NetBSD: rf_netbsdkintf.c,v 1.195 2006/01/08 09:09:53 yamt Exp $"); #include <sys/param.h> #include <sys/errno.h> @@ -983,8 +983,6 @@ raidioctl(dev_t dev, u_long cmd, caddr_t data, int flag, struct lwp *l) if (clabel == NULL) return (ENOMEM); - memset((char *) clabel, 0, sizeof(RF_ComponentLabel_t)); - retcode = copyin( *clabel_ptr, clabel, sizeof(RF_ComponentLabel_t)); @@ -1003,12 +1001,14 @@ raidioctl(dev_t dev, u_long cmd, caddr_t data, int flag, struct lwp *l) return(EINVAL); } - raidread_component_label(raidPtr->Disks[column].dev, + retcode = raidread_component_label(raidPtr->Disks[column].dev, raidPtr->raid_cinfo[column].ci_vp, clabel ); - retcode = copyout(clabel, *clabel_ptr, - sizeof(RF_ComponentLabel_t)); + if (retcode == 0) { + retcode = copyout(clabel, *clabel_ptr, + sizeof(RF_ComponentLabel_t)); + } RF_Free(clabel, sizeof(RF_ComponentLabel_t)); return (retcode); |
