[Cluster-devel] [PATCH] dlm: Fix uninitialised variable in receiving

[Robert Peterson <rpeterso@redhat.com>]

Patrick Caulfield wrote:
> The length of the second element of the kvec array was not initialised before
> being added to the first one. This could cause invalid lengths to be passed to
> kernel_recvmsg
> 
> Signed-Off-By: Patrick Caulfield <pcaulfie@redhat.com>
> 
> 
> ------------------------------------------------------------------------
> 
> diff --git a/fs/dlm/lowcomms-tcp.c b/fs/dlm/lowcomms-tcp.c
> index 07e0a12..9bfe7fb 100644
> --- a/fs/dlm/lowcomms-tcp.c
> +++ b/fs/dlm/lowcomms-tcp.c
> @@ -299,6 +299,7 @@ static int receive_from_sock(struct connection *con)
>  	 */
>  	iov[0].iov_len = con->cb.base - cbuf_data(&con->cb);
>  	iov[0].iov_base = page_address(con->rx_page) + cbuf_data(&con->cb);
> +	iov[1].iov_len = 0;
>  	nvec = 1;
>  
>  	/*
ACK

This is a necessary fix.

Bob Peterson
Red Hat Cluster Suite