Re: [PATCH 1/1] CIFS: add int cast to silence warning

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

 



I merged this with your previous patch into one and pushed to
cifs-2.6.git for-next

On Mon, Jun 26, 2017 at 7:49 AM, Aurelien Aptel <aaptel@xxxxxxxx> wrote:
> The '*' field specifier expects an int and sizeof returns an unsigned
> long int but its always going to be a small positive value so casting is
> safe.
>
> Signed-off-by: Aurelien Aptel <aaptel@xxxxxxxx>
> ---
>  fs/cifs/smb2transport.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/fs/cifs/smb2transport.c b/fs/cifs/smb2transport.c
> index 437d11711b3e..ad0492c682e7 100644
> --- a/fs/cifs/smb2transport.c
> +++ b/fs/cifs/smb2transport.c
> @@ -349,7 +349,8 @@ generate_smb3signingkey(struct cifs_ses *ses,
>          * print it as a long long. we dump it as we got it on the
>          * wire.
>          */
> -       cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
> +       cifs_dbg(VFS, "Session Id    %*ph\n", (int)sizeof(ses->Suid),
> +                &ses->Suid);
>         cifs_dbg(VFS, "Session Key   %*ph\n",
>                  SMB2_NTLMV2_SESSKEY_SIZE, ses->auth_key.response);
>         cifs_dbg(VFS, "Signing Key   %*ph\n",
> --
> 2.12.3
>



-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux