Commit 8ef32967 authored by Dan Carpenter's avatar Dan Carpenter Committed by Steve French
Browse files

cifsd: fix a IS_ERR() vs NULL bug 



The smb_direct_alloc_sendmsg() function never returns NULL, it only
returns error pointers so the check needs to be updated.

Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarSergey Senozhatsky <sergey.senozhatsky@gmail.com>
Signed-off-by: default avatarNamjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent 86df49e1

fs/cifsd/transport_rdma.c

+2 −2
Original line number Diff line number Diff line
@@ -997,8 +997,8 @@ static int smb_direct_create_header(struct smb_direct_transport *t, 
	int ret;



	sendmsg = smb_direct_alloc_sendmsg(t);

	if (!sendmsg)

		return -ENOMEM;

	if (IS_ERR(sendmsg))

		return PTR_ERR(sendmsg);



	/* Fill in the packet header */

	packet = (struct smb_direct_data_transfer *)sendmsg->packet;