| 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
| ||
|
Date: Mon, 28 Jun 2021 07:25:29 +0800
From: kernel test robot <lkp@...el.com>
To: Nick Desaulniers <ndesaulniers@...gle.com>
Cc: kbuild-all@...ts.01.org, clang-built-linux@...glegroups.com,
linux-kernel@...r.kernel.org,
Andrew Morton <akpm@...ux-foundation.org>,
Linux Memory Management List <linux-mm@...ck.org>
Subject: arch/powerpc/boot/decompress.c:133:8: warning: implicit declaration
of function '__decompress' is invalid in C99
Hi Nick,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 62fb9874f5da54fdb243003b386128037319b219
commit: 1e1b6d63d6340764e00356873e5794225a2a03ea lib/string.c: implement stpcpy
date: 9 months ago
config: powerpc64-randconfig-r003-20210628 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 59558129276098d62046c8cda92240d292cbfb1c)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc64 cross compiling tool for clang build
# apt-get install binutils-powerpc64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1e1b6d63d6340764e00356873e5794225a2a03ea
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 1e1b6d63d6340764e00356873e5794225a2a03ea
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
>> arch/powerpc/boot/decompress.c:133:8: warning: implicit declaration of function '__decompress' is invalid in C99 [-Wimplicit-function-declaration]
ret = __decompress(inbuf, input_size, NULL, flush, outbuf,
^
1 warning generated.
vim +/__decompress +133 arch/powerpc/boot/decompress.c
1b7898ee276b39 Oliver O'Halloran 2016-09-22 98
1b7898ee276b39 Oliver O'Halloran 2016-09-22 99 /**
1b7898ee276b39 Oliver O'Halloran 2016-09-22 100 * partial_decompress - decompresses part or all of a compressed buffer
1b7898ee276b39 Oliver O'Halloran 2016-09-22 101 * @inbuf: input buffer
1b7898ee276b39 Oliver O'Halloran 2016-09-22 102 * @input_size: length of the input buffer
1b7898ee276b39 Oliver O'Halloran 2016-09-22 103 * @outbuf: input buffer
1b7898ee276b39 Oliver O'Halloran 2016-09-22 104 * @output_size: length of the input buffer
1b7898ee276b39 Oliver O'Halloran 2016-09-22 105 * @skip number of output bytes to ignore
1b7898ee276b39 Oliver O'Halloran 2016-09-22 106 *
1b7898ee276b39 Oliver O'Halloran 2016-09-22 107 * This function takes compressed data from inbuf, decompresses and write it to
1b7898ee276b39 Oliver O'Halloran 2016-09-22 108 * outbuf. Once output_size bytes are written to the output buffer, or the
1b7898ee276b39 Oliver O'Halloran 2016-09-22 109 * stream is exhausted the function will return the number of bytes that were
1b7898ee276b39 Oliver O'Halloran 2016-09-22 110 * decompressed. Otherwise it will return whatever error code the decompressor
1b7898ee276b39 Oliver O'Halloran 2016-09-22 111 * reported (NB: This is specific to each decompressor type).
1b7898ee276b39 Oliver O'Halloran 2016-09-22 112 *
1b7898ee276b39 Oliver O'Halloran 2016-09-22 113 * The skip functionality is mainly there so the program and discover
1b7898ee276b39 Oliver O'Halloran 2016-09-22 114 * the size of the compressed image so that it can ask firmware (if present)
1b7898ee276b39 Oliver O'Halloran 2016-09-22 115 * for an appropriately sized buffer.
1b7898ee276b39 Oliver O'Halloran 2016-09-22 116 */
1b7898ee276b39 Oliver O'Halloran 2016-09-22 117 long partial_decompress(void *inbuf, unsigned long input_size,
1b7898ee276b39 Oliver O'Halloran 2016-09-22 118 void *outbuf, unsigned long output_size, unsigned long _skip)
1b7898ee276b39 Oliver O'Halloran 2016-09-22 119 {
1b7898ee276b39 Oliver O'Halloran 2016-09-22 120 int ret;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 121
1b7898ee276b39 Oliver O'Halloran 2016-09-22 122 /*
1b7898ee276b39 Oliver O'Halloran 2016-09-22 123 * The skipped bytes needs to be included in the size of data we want
1b7898ee276b39 Oliver O'Halloran 2016-09-22 124 * to decompress.
1b7898ee276b39 Oliver O'Halloran 2016-09-22 125 */
1b7898ee276b39 Oliver O'Halloran 2016-09-22 126 output_size += _skip;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 127
1b7898ee276b39 Oliver O'Halloran 2016-09-22 128 decompressed_bytes = 0;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 129 output_buffer = outbuf;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 130 limit = output_size;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 131 skip = _skip;
1b7898ee276b39 Oliver O'Halloran 2016-09-22 132
1b7898ee276b39 Oliver O'Halloran 2016-09-22 @133 ret = __decompress(inbuf, input_size, NULL, flush, outbuf,
:::::: The code at line 133 was first introduced by commit
:::::: 1b7898ee276b39e54d870dc4ef3374f663d0b426 powerpc/boot: Use the pre-boot decompression API
:::::: TO: Oliver O'Halloran <oohall@...il.com>
:::::: CC: Michael Ellerman <mpe@...erman.id.au>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Download attachment ".config.gz" of type "application/gzip" (33834 bytes)
Powered by blists - more mailing lists