| From 9dd6a06430c94299651d74b9ed5ca8396ab8ff1f Mon Sep 17 00:00:00 2001 |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| Date: Sat, 1 Aug 2015 16:08:17 -0700 |
| Subject: hv: util: checking the wrong variable |
| |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| |
| commit 9dd6a06430c94299651d74b9ed5ca8396ab8ff1f upstream. |
| |
| We don't catch this allocation failure because there is a typo and we |
| check the wrong variable. |
| |
| Fixes: 14b50f80c32d ('Drivers: hv: util: introduce hv_utils_transport abstraction') |
| |
| Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> |
| Reviewed-by: Vitaly Kuznetsov <vkuznets@redhat.com> |
| Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/hv/hv_utils_transport.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/hv/hv_utils_transport.c |
| +++ b/drivers/hv/hv_utils_transport.c |
| @@ -186,7 +186,7 @@ int hvutil_transport_send(struct hvutil_ |
| return -EINVAL; |
| } else if (hvt->mode == HVUTIL_TRANSPORT_NETLINK) { |
| cn_msg = kzalloc(sizeof(*cn_msg) + len, GFP_ATOMIC); |
| - if (!msg) |
| + if (!cn_msg) |
| return -ENOMEM; |
| cn_msg->id.idx = hvt->cn_id.idx; |
| cn_msg->id.val = hvt->cn_id.val; |