diff options
author | K. Y. Srinivasan <kys@microsoft.com> | 2014-08-27 16:25:35 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-10-30 09:35:11 -0700 |
commit | bd6174827fa6862fc2b59c2e0eeee5e5329befa2 (patch) | |
tree | 0809dd25853d4b6996d967b20a56c2337e115c73 /drivers/hv | |
parent | b5e03dd7bfe9a3a9cdeb160f869b3c264d51706f (diff) | |
download | lwn-bd6174827fa6862fc2b59c2e0eeee5e5329befa2.tar.gz lwn-bd6174827fa6862fc2b59c2e0eeee5e5329befa2.zip |
Drivers: hv: vmbus: Fix a bug in vmbus_open()
commit 45d727cee9e200f5b351528b9fb063b69cf702c8 upstream.
Fix a bug in vmbus_open() and properly propagate the error. I would
like to thank Dexuan Cui <decui@microsoft.com> for identifying the
issue.
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Tested-by: Sitsofe Wheeler <sitsofe@yahoo.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hv')
-rw-r--r-- | drivers/hv/channel.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index d34db9fdc518..92f34de7aee9 100644 --- a/drivers/hv/channel.c +++ b/drivers/hv/channel.c @@ -199,8 +199,10 @@ int vmbus_open(struct vmbus_channel *newchannel, u32 send_ringbuffer_size, ret = vmbus_post_msg(open_msg, sizeof(struct vmbus_channel_open_channel)); - if (ret != 0) + if (ret != 0) { + err = ret; goto error1; + } t = wait_for_completion_timeout(&open_info->waitevent, 5*HZ); if (t == 0) { |