Re: [PATCH] clnt_vc_control: fix VERS/PROG size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 16/05/13 16:25, Jan Stancek wrote:
> Following commit converted some operations to use memcpy, but the size
> of copied areas is now u_long, previously it was u_int32_t:
>   commit fac3eb51c7055055bdd7dccd9db556d9e4a1cf35
>   Author: Steve Dickson <steved@xxxxxxxxxx>
>   Date:   Tue Feb 12 14:26:31 2013 -0500
>       clnt_vc_control: Removed a strict-aliasing warning
> 
> This broke mount on ppc64/s390x:
>   # ./mount.nfs -vvv -onfsvers=3,tcp rhel6-nfs:/export/home /mnt/test
>   mount.nfs: timeout set for Fri May  3 15:20:14 2013
>   mount.nfs: trying text-based options 'nfsvers=3,tcp,addr=10.1.1.5'
>   mount.nfs: prog 100003, trying vers=3, prot=6
>   mount.nfs: portmap query failed: RPC: Program/version mismatch
>   mount.nfs: Protocol not supported
> 
> Signed-off-by: Jan Stancek <jstancek@xxxxxxxxxx>
Committed....

steved.

> ---
>  src/clnt_vc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/clnt_vc.c b/src/clnt_vc.c
> index 7769809..2eab9e4 100644
> --- a/src/clnt_vc.c
> +++ b/src/clnt_vc.c
> @@ -505,7 +505,7 @@ clnt_vc_control(cl, request, info)
>  	sigset_t newmask;
>  	int rpc_lock_value;
>  	u_int32_t tmp;
> -	u_long ltmp;
> +	u_int32_t ltmp;
>  
>  	assert(cl != NULL);
>  
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux