Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1612357 > unrolled thread
| Started by | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| First post | 2017-03-29 23:20 +0200 |
| Last post | 2017-03-31 13:40 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] initify_plugin: one more warning workaround Arnd Bergmann <arnd@arndb.de> - 2017-03-29 23:20 +0200
Re: [PATCH] initify_plugin: one more warning workaround kbuild test robot <lkp@intel.com> - 2017-03-31 12:40 +0200
Re: [PATCH] initify_plugin: one more warning workaround Arnd Bergmann <arnd@arndb.de> - 2017-03-31 13:30 +0200
Re: [kbuild-all] [PATCH] initify_plugin: one more warning workaround Fengguang Wu <lkp@intel.com> - 2017-03-31 13:40 +0200
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2017-03-29 23:20 +0200 |
| Subject | [PATCH] initify_plugin: one more warning workaround |
| Message-ID | <tqtay-4hp-29@gated-at.bofh.it> |
lib/vsprintf.c: In function 'bstr_printf':
lib/vsprintf.c:2409:5: error: 'bstr_printf' captures its 3 ('fmt') parameter, please remove it from the nocapture attribute. [-Werror]
int bstr_printf(char *buf, size_t size, const char *fmt, const u32 *bin_buf)
Cc: Kees Cook <keescook@chromium.org>
Cc: Emese Revfy <re.emese@gmail.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
lib/vsprintf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/vsprintf.c b/lib/vsprintf.c
index 21dbdf05dfdf..8e4725341e93 100644
--- a/lib/vsprintf.c
+++ b/lib/vsprintf.c
@@ -2406,7 +2406,7 @@ EXPORT_SYMBOL_GPL(vbin_printf);
* return is greater than or equal to @size, the resulting
* string is truncated.
*/
-int bstr_printf(char *buf, size_t size, const char *fmt, const u32 *bin_buf)
+__unverified_nocapture(3) int bstr_printf(char *buf, size_t size, const char *fmt, const u32 *bin_buf)
{
struct printf_spec spec = {0};
char *str, *end;
--
2.9.0
[toc] | [next] | [standalone]
| From | kbuild test robot <lkp@intel.com> |
|---|---|
| Date | 2017-03-31 12:40 +0200 |
| Message-ID | <tr28i-3pV-9@gated-at.bofh.it> |
| In reply to | #1612357 |
[Multipart message — attachments visible in raw view] — view raw
Hi Arnd,
[auto build test WARNING on linus/master]
[also build test WARNING on v4.11-rc4 next-20170330]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/initify_plugin-one-more-warning-workaround/20170331-161135
reproduce: make htmldocs
All warnings (new ones prefixed by >>):
>> lib/vsprintf.c:2407: warning: No description found for parameter '3'
>> lib/vsprintf.c:2407: warning: Excess function parameter 'buf' description in '__unverified_nocapture'
>> lib/vsprintf.c:2407: warning: Excess function parameter 'size' description in '__unverified_nocapture'
>> lib/vsprintf.c:2407: warning: Excess function parameter 'fmt' description in '__unverified_nocapture'
>> lib/vsprintf.c:2407: warning: Excess function parameter 'bin_buf' description in '__unverified_nocapture'
lib/crc32.c:148: warning: No description found for parameter 'tab)[256]'
lib/crc32.c:148: warning: Excess function parameter 'tab' description in 'crc32_le_generic'
lib/crc32.c:293: warning: No description found for parameter 'tab)[256]'
lib/crc32.c:293: warning: Excess function parameter 'tab' description in 'crc32_be_generic'
lib/crc32.c:1: warning: no structured comments found
vim +/3 +2407 lib/vsprintf.c
4370aa4a Lai Jiangshan 2009-03-06 2391 * This function like C99 vsnprintf, but the difference is that vsnprintf gets
4370aa4a Lai Jiangshan 2009-03-06 2392 * arguments from stack, and bstr_printf gets arguments from @bin_buf which is
4370aa4a Lai Jiangshan 2009-03-06 2393 * a binary buffer that generated by vbin_printf.
4370aa4a Lai Jiangshan 2009-03-06 2394 *
4370aa4a Lai Jiangshan 2009-03-06 2395 * The format follows C99 vsnprintf, but has some extensions:
0efb4d20 Steven Rostedt 2009-09-17 2396 * see vsnprintf comment for details.
4370aa4a Lai Jiangshan 2009-03-06 2397 *
4370aa4a Lai Jiangshan 2009-03-06 2398 * The return value is the number of characters which would
4370aa4a Lai Jiangshan 2009-03-06 2399 * be generated for the given input, excluding the trailing
4370aa4a Lai Jiangshan 2009-03-06 2400 * '\0', as per ISO C99. If you want to have the exact
4370aa4a Lai Jiangshan 2009-03-06 2401 * number of characters written into @buf as return value
4370aa4a Lai Jiangshan 2009-03-06 2402 * (not including the trailing '\0'), use vscnprintf(). If the
4370aa4a Lai Jiangshan 2009-03-06 2403 * return is greater than or equal to @size, the resulting
4370aa4a Lai Jiangshan 2009-03-06 2404 * string is truncated.
4370aa4a Lai Jiangshan 2009-03-06 2405 */
85dcce6c Arnd Bergmann 2017-03-29 2406 __unverified_nocapture(3) int bstr_printf(char *buf, size_t size, const char *fmt, const u32 *bin_buf)
4370aa4a Lai Jiangshan 2009-03-06 @2407 {
fef20d9c Frederic Weisbecker 2009-03-06 2408 struct printf_spec spec = {0};
d4be151b André Goddard Rosa 2009-12-14 2409 char *str, *end;
d4be151b André Goddard Rosa 2009-12-14 2410 const char *args = (const char *)bin_buf;
4370aa4a Lai Jiangshan 2009-03-06 2411
762abb51 Rasmus Villemoes 2015-11-06 2412 if (WARN_ON_ONCE(size > INT_MAX))
4370aa4a Lai Jiangshan 2009-03-06 2413 return 0;
4370aa4a Lai Jiangshan 2009-03-06 2414
4370aa4a Lai Jiangshan 2009-03-06 2415 str = buf;
:::::: The code at line 2407 was first introduced by commit
:::::: 4370aa4aa75391a5e2e06bccb0919109f725ed8e vsprintf: add binary printf
:::::: TO: Lai Jiangshan <laijs@cn.fujitsu.com>
:::::: CC: Ingo Molnar <mingo@elte.hu>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[toc] | [prev] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2017-03-31 13:30 +0200 |
| Message-ID | <tr2UH-3Xe-53@gated-at.bofh.it> |
| In reply to | #1613853 |
On Fri, Mar 31, 2017 at 12:33 PM, kbuild test robot <lkp@intel.com> wrote:
> Hi Arnd,
>
> [auto build test WARNING on linus/master]
> [also build test WARNING on v4.11-rc4 next-20170330]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
I expected it to work on next but not on linus/master
> url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/initify_plugin-one-more-warning-workaround/20170331-161135
> reproduce: make htmldocs
>
> All warnings (new ones prefixed by >>):
>
>>> lib/vsprintf.c:2407: warning: No description found for parameter '3'
>>> lib/vsprintf.c:2407: warning: Excess function parameter 'buf' description in '__unverified_nocapture'
>>> lib/vsprintf.c:2407: warning: Excess function parameter 'size' description in '__unverified_nocapture'
>>> lib/vsprintf.c:2407: warning: Excess function parameter 'fmt' description in '__unverified_nocapture'
>>> lib/vsprintf.c:2407: warning: Excess function parameter 'bin_buf' description in '__unverified_nocapture'
These warnings will definitely show up on mainline today.
> lib/crc32.c:148: warning: No description found for parameter 'tab)[256]'
> lib/crc32.c:148: warning: Excess function parameter 'tab' description in 'crc32_le_generic'
> lib/crc32.c:293: warning: No description found for parameter 'tab)[256]'
> lib/crc32.c:293: warning: Excess function parameter 'tab' description in 'crc32_be_generic'
> lib/crc32.c:1: warning: no structured comments found
I don't see how this output relates to my patch.
Did only the latter appear on linux-next?
Arnd
[toc] | [prev] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2017-03-31 13:40 +0200 |
| Subject | Re: [kbuild-all] [PATCH] initify_plugin: one more warning workaround |
| Message-ID | <tr34m-40O-33@gated-at.bofh.it> |
| In reply to | #1613903 |
On Fri, Mar 31, 2017 at 01:20:04PM +0200, Arnd Bergmann wrote: >On Fri, Mar 31, 2017 at 12:33 PM, kbuild test robot <lkp@intel.com> wrote: >> Hi Arnd, >> >> [auto build test WARNING on linus/master] >> [also build test WARNING on v4.11-rc4 next-20170330] >> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > >I expected it to work on next but not on linus/master > >> url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/initify_plugin-one-more-warning-workaround/20170331-161135 >> reproduce: make htmldocs >> >> All warnings (new ones prefixed by >>): >> >>>> lib/vsprintf.c:2407: warning: No description found for parameter '3' >>>> lib/vsprintf.c:2407: warning: Excess function parameter 'buf' description in '__unverified_nocapture' >>>> lib/vsprintf.c:2407: warning: Excess function parameter 'size' description in '__unverified_nocapture' >>>> lib/vsprintf.c:2407: warning: Excess function parameter 'fmt' description in '__unverified_nocapture' >>>> lib/vsprintf.c:2407: warning: Excess function parameter 'bin_buf' description in '__unverified_nocapture' > >These warnings will definitely show up on mainline today. > >> lib/crc32.c:148: warning: No description found for parameter 'tab)[256]' >> lib/crc32.c:148: warning: Excess function parameter 'tab' description in 'crc32_le_generic' >> lib/crc32.c:293: warning: No description found for parameter 'tab)[256]' >> lib/crc32.c:293: warning: Excess function parameter 'tab' description in 'crc32_be_generic' >> lib/crc32.c:1: warning: no structured comments found > >I don't see how this output relates to my patch. They are old warnings (not prefixed by >>), so not related to your patch. >Did only the latter appear on linux-next? Since your patch is firstly applied to mainline kernel, the error messages should correspond to mainline kernel. Regards, Fengguang
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web