diff options
author | Bill Pemberton <wfp5p@virginia.edu> | 2009-07-28 13:46:22 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-09-15 12:01:52 -0700 |
commit | 2295ba2e7c4141e864f66830b07e635245233a3d (patch) | |
tree | 171fce7ebd2a1a770b98b7825d0d31d5b76abccc /drivers/staging/hv/storvsc_drv.c | |
parent | 3d3b5518a35fa3c31b74224d944cbc83302c5ab4 (diff) | |
download | kernel_samsung_smdk4412-2295ba2e7c4141e864f66830b07e635245233a3d.zip kernel_samsung_smdk4412-2295ba2e7c4141e864f66830b07e635245233a3d.tar.gz kernel_samsung_smdk4412-2295ba2e7c4141e864f66830b07e635245233a3d.tar.bz2 |
Staging: hv: check return value of driver_for_each_device()
The return value of driver_for_each_device() is now checked. A
non-zero value simply generates a warning message, but it's better
than not checking at all.
Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/hv/storvsc_drv.c')
-rw-r--r-- | drivers/staging/hv/storvsc_drv.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/staging/hv/storvsc_drv.c b/drivers/staging/hv/storvsc_drv.c index 385e84b..120ec52 100644 --- a/drivers/staging/hv/storvsc_drv.c +++ b/drivers/staging/hv/storvsc_drv.c @@ -188,8 +188,8 @@ void storvsc_drv_exit(void) { STORVSC_DRIVER_OBJECT *storvsc_drv_obj=&g_storvsc_drv.drv_obj; struct driver_context *drv_ctx=&g_storvsc_drv.drv_ctx; - struct device *current_dev=NULL; + int ret; DPRINT_ENTER(STORVSC_DRV); @@ -198,7 +198,13 @@ void storvsc_drv_exit(void) current_dev = NULL; /* Get the device */ - driver_for_each_device(&drv_ctx->driver, NULL, (void*)¤t_dev, storvsc_drv_exit_cb); + ret = driver_for_each_device(&drv_ctx->driver, NULL, + (void *) ¤t_dev, + storvsc_drv_exit_cb); + + if (ret) + DPRINT_WARN(STORVSC_DRV, + "driver_for_each_device returned %d", ret); if (current_dev == NULL) break; |