Commit fe274c5a authored by Insu Yun's avatar Insu Yun Committed by Doug Ledford

usnic: correctly handle kzalloc return value

Since kzalloc returns memory address, not error code,
it should be checked whether it is null or not.
Signed-off-by: default avatarInsu Yun <wuninsu@gmail.com>
Reviewed-by: default avatarDave Goodell <dgoodell@cisco.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent 35ef4a9e
...@@ -236,8 +236,8 @@ create_roce_custom_flow(struct usnic_ib_qp_grp *qp_grp, ...@@ -236,8 +236,8 @@ create_roce_custom_flow(struct usnic_ib_qp_grp *qp_grp,
/* Create Flow Handle */ /* Create Flow Handle */
qp_flow = kzalloc(sizeof(*qp_flow), GFP_ATOMIC); qp_flow = kzalloc(sizeof(*qp_flow), GFP_ATOMIC);
if (IS_ERR_OR_NULL(qp_flow)) { if (!qp_flow) {
err = qp_flow ? PTR_ERR(qp_flow) : -ENOMEM; err = -ENOMEM;
goto out_dealloc_flow; goto out_dealloc_flow;
} }
qp_flow->flow = flow; qp_flow->flow = flow;
...@@ -311,8 +311,8 @@ create_udp_flow(struct usnic_ib_qp_grp *qp_grp, ...@@ -311,8 +311,8 @@ create_udp_flow(struct usnic_ib_qp_grp *qp_grp,
/* Create qp_flow */ /* Create qp_flow */
qp_flow = kzalloc(sizeof(*qp_flow), GFP_ATOMIC); qp_flow = kzalloc(sizeof(*qp_flow), GFP_ATOMIC);
if (IS_ERR_OR_NULL(qp_flow)) { if (!qp_flow) {
err = qp_flow ? PTR_ERR(qp_flow) : -ENOMEM; err = -ENOMEM;
goto out_dealloc_flow; goto out_dealloc_flow;
} }
qp_flow->flow = flow; qp_flow->flow = flow;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment