[PATCH] CIFS: convert to use le32_add_cpu()

Shirish Pargaonkar shirishpargaonkar at gmail.com
Wed Aug 28 07:11:40 MDT 2013


Looks correct.

Acked-by: Shirish Pargaonkar <shirishpargaonkar at gmail.com>

On Mon, Aug 26, 2013 at 1:34 AM, Wei Yongjun <weiyj.lk at gmail.com> wrote:
> From: Wei Yongjun <yongjun_wei at trendmicro.com.cn>
>
> Convert cpu_to_le32(le32_to_cpu(E1) + E2) to use le32_add_cpu().
>
> Signed-off-by: Wei Yongjun <yongjun_wei at trendmicro.com.cn>
> ---
>  fs/cifs/smb2pdu.c | 8 ++------
>  1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c
> index abc9c28..4ce66bc 100644
> --- a/fs/cifs/smb2pdu.c
> +++ b/fs/cifs/smb2pdu.c
> @@ -939,9 +939,7 @@ add_lease_context(struct kvec *iov, unsigned int *num_iovec, __u8 *oplock)
>                 req->CreateContextsOffset = cpu_to_le32(
>                                 sizeof(struct smb2_create_req) - 4 +
>                                 iov[num - 1].iov_len);
> -       req->CreateContextsLength = cpu_to_le32(
> -                               le32_to_cpu(req->CreateContextsLength) +
> -                               sizeof(struct create_lease));
> +       le32_add_cpu(&req->CreateContextsLength, sizeof(struct create_lease));
>         inc_rfc1001_len(&req->hdr, sizeof(struct create_lease));
>         *num_iovec = num + 1;
>         return 0;
> @@ -967,9 +965,7 @@ add_durable_context(struct kvec *iov, unsigned int *num_iovec,
>                 req->CreateContextsOffset =
>                         cpu_to_le32(sizeof(struct smb2_create_req) - 4 +
>                                                                 iov[1].iov_len);
> -       req->CreateContextsLength =
> -                       cpu_to_le32(le32_to_cpu(req->CreateContextsLength) +
> -                                               sizeof(struct create_durable));
> +       le32_add_cpu(&req->CreateContextsLength, sizeof(struct create_durable));
>         inc_rfc1001_len(&req->hdr, sizeof(struct create_durable));
>         *num_iovec = num + 1;
>         return 0;
>


More information about the samba-technical mailing list