[Linux-cluster] [PATCH] dlm: fix missing endian conversion of rcom_status flags

Bob Peterson rpeterso at redhat.com
Tue Oct 14 20:22:36 UTC 2014


----- Original Message -----
> The flags are already converted to le when being sent,
> but are not being converted back to cpu when received.
> 
> Signed-off-by: Neale Ferguson <neale at sinenomine.net>
> Signed-off-by: David Teigland <teigland at redhat.com>
> ---
>  fs/dlm/rcom.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/dlm/rcom.c b/fs/dlm/rcom.c
> index 9d61947d473a..f3f5e72a29ba 100644
> --- a/fs/dlm/rcom.c
> +++ b/fs/dlm/rcom.c
> @@ -206,7 +206,7 @@ static void receive_rcom_status(struct dlm_ls *ls, struct
> dlm_rcom *rc_in)
>  
>  	rs = (struct rcom_status *)rc_in->rc_buf;
>  
> -	if (!(rs->rs_flags & DLM_RSF_NEED_SLOTS)) {
> +	if (!(le32_to_cpu(rs->rs_flags) & DLM_RSF_NEED_SLOTS)) {
>  		status = dlm_recover_status(ls);
>  		goto do_create;
>  	}
> --
> 1.8.3.1
> 
> 
> --
> Linux-cluster mailing list
> Linux-cluster at redhat.com
> https://www.redhat.com/mailman/listinfo/linux-cluster

Hi Dave,

Did you mean for this patch to go to cluster-devel?

Bob Peterson
Red Hat File Systems




More information about the Linux-cluster mailing list