| From 9435330bd885600bb666af6bb111834a8e475ba5 Mon Sep 17 00:00:00 2001 |
| From: Olaf Hering <olaf@aepfle.de> |
| Date: Mon, 14 Dec 2015 16:01:36 -0800 |
| Subject: [PATCH 037/135] Drivers: hv: util: catch allocation errors |
| |
| [ Upstream commit cdc0c0c94e4e6dfa371d497a3130f83349b6ead6 ] |
| |
| Catch allocation errors in hvutil_transport_send. |
| |
| Fixes: 14b50f80c32d ('Drivers: hv: util: introduce hv_utils_transport abstraction') |
| |
| Signed-off-by: Olaf Hering <olaf@aepfle.de> |
| Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| Signed-off-by: Sasha Levin <alexander.levin@verizon.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/hv/hv_utils_transport.c | 9 ++++++--- |
| 1 file changed, 6 insertions(+), 3 deletions(-) |
| |
| --- a/drivers/hv/hv_utils_transport.c |
| +++ b/drivers/hv/hv_utils_transport.c |
| @@ -204,9 +204,12 @@ int hvutil_transport_send(struct hvutil_ |
| goto out_unlock; |
| } |
| hvt->outmsg = kzalloc(len, GFP_KERNEL); |
| - memcpy(hvt->outmsg, msg, len); |
| - hvt->outmsg_len = len; |
| - wake_up_interruptible(&hvt->outmsg_q); |
| + if (hvt->outmsg) { |
| + memcpy(hvt->outmsg, msg, len); |
| + hvt->outmsg_len = len; |
| + wake_up_interruptible(&hvt->outmsg_q); |
| + } else |
| + ret = -ENOMEM; |
| out_unlock: |
| mutex_unlock(&hvt->outmsg_lock); |
| return ret; |