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 -- 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