linux-kernel - Re: linux-next: build warning after merge of the cifs tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 26 Jun 2017 11:57:46 -0500
From:   Steve French <smfrench@...il.com>
To:     Stephen Rothwell <sfr@...b.auug.org.au>
Cc:     CIFS <linux-cifs@...r.kernel.org>,
        Linux-Next Mailing List <linux-next@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Aurélien Aptel <aaptel@...e.com>
Subject: Re: linux-next: build warning after merge of the cifs tree

I just merged an updated version of this patch to remove the warning
to cifs-2.6.git for-next

On Sun, Jun 25, 2017 at 7:07 PM, Stephen Rothwell <sfr@...b.auug.org.au> wrote:
> Hi all,
>
> After merging the cifs tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> In file included from include/linux/kernel.h:13:0,
>                  from include/linux/list.h:8,
>                  from include/linux/wait.h:6,
>                  from include/linux/fs.h:5,
>                  from fs/cifs/smb2transport.c:25:
> fs/cifs/smb2transport.c: In function 'generate_smb3signingkey':
> fs/cifs/cifs_debug.h:55:24: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
>    pr_debug_ratelimited("%s: "    \
>                         ^
> include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
>  #define pr_fmt(fmt) fmt
>                      ^
> fs/cifs/cifs_debug.h:55:3: note: in expansion of macro 'pr_debug_ratelimited'
>    pr_debug_ratelimited("%s: "    \
>    ^
> fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>   ^
> In file included from fs/cifs/smb2transport.c:39:0:
> fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 2 has type 'long unsigned int' [-Wformat=]
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>                 ^
> fs/cifs/cifs_debug.h:58:16: note: in definition of macro 'cifs_dbg'
>    cifs_vfs_err(fmt, ##__VA_ARGS__);   \
>                 ^
> In file included from include/linux/kernel.h:13:0,
>                  from include/linux/list.h:8,
>                  from include/linux/wait.h:6,
>                  from include/linux/fs.h:5,
>                  from fs/cifs/smb2transport.c:25:
> fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>                 ^
> include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
>  #define pr_fmt(fmt) fmt
>                      ^
> fs/cifs/cifs_debug.h:60:3: note: in expansion of macro 'pr_debug_ratelimited'
>    pr_debug_ratelimited(fmt, ##__VA_ARGS__);  \
>    ^
> fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>   ^
>
> Introduced by commit
>
>   ba4f2a8b205f ("CIFS: add CONFIG_CIFS_DEBUG_KEYS to dump encryption keys")
>
> --
> Cheers,
> Stephen Rothwell



-- 
Thanks,

Steve