From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> Fix to return -EINVAL in the version check error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> --- drivers/hv/connection.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/hv/connection.c b/drivers/hv/connection.c index 8f4743a..b3eb50f 100644 --- a/drivers/hv/connection.c +++ b/drivers/hv/connection.c @@ -204,8 +204,10 @@ int vmbus_connect(void) version = vmbus_get_next_version(version); } while (version != VERSION_INVAL); - if (version == VERSION_INVAL) + if (version == VERSION_INVAL) { + ret = -EINVAL; goto cleanup; + } vmbus_proto_version = version; pr_info("Hyper-V Host Build:%d-%d.%d-%d-%d.%d; Vmbus version:%d.%d\n", _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel