BCC: [email protected] CC: [email protected] In-Reply-To: <[email protected]> References: <[email protected]> TO: Dmitry Safonov <[email protected]> TO: Eric Dumazet <[email protected]> TO: "David S. Miller" <[email protected]> CC: [email protected] TO: [email protected] CC: Dmitry Safonov <[email protected]> CC: Andy Lutomirski <[email protected]> CC: Ard Biesheuvel <[email protected]> CC: Bob Gilligan <[email protected]> CC: David Ahern <[email protected]> CC: Eric Biggers <[email protected]> CC: Francesco Ruggeri <[email protected]> CC: Herbert Xu <[email protected]> CC: Hideaki YOSHIFUJI <[email protected]> CC: Ivan Delalande <[email protected]> CC: Jakub Kicinski <[email protected]> CC: Leonard Crestez <[email protected]> CC: Paolo Abeni <[email protected]> CC: Salam Noureddine <[email protected]> CC: Shuah Khan <[email protected]> CC: [email protected]
Hi Dmitry, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on e34cfee65ec891a319ce79797dda18083af33a76] url: https://github.com/intel-lab-lkp/linux/commits/Dmitry-Safonov/net-tcp-Add-TCP-AO-support/20220819-010628 base: e34cfee65ec891a319ce79797dda18083af33a76 :::::: branch date: 4 days ago :::::: commit date: 4 days ago config: x86_64-randconfig-m001 (https://download.01.org/0day-ci/archive/20220822/[email protected]/config) compiler: gcc-11 (Debian 11.3.0-5) 11.3.0 If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> New smatch warnings: crypto/crypto_pool.c:203 crypto_pool_alloc_ahash() error: uninitialized symbol 'err'. Old smatch warnings: crypto/crypto_pool.c:193 crypto_pool_alloc_ahash() error: testing array offset 'i' after use. vim +/err +203 crypto/crypto_pool.c f4c3873630fc8c4 Dmitry Safonov 2022-08-18 166 f4c3873630fc8c4 Dmitry Safonov 2022-08-18 167 /** f4c3873630fc8c4 Dmitry Safonov 2022-08-18 168 * crypto_pool_alloc_ahash - allocates pool for ahash requests f4c3873630fc8c4 Dmitry Safonov 2022-08-18 169 * @alg: name of async hash algorithm f4c3873630fc8c4 Dmitry Safonov 2022-08-18 170 */ f4c3873630fc8c4 Dmitry Safonov 2022-08-18 171 int crypto_pool_alloc_ahash(const char *alg) f4c3873630fc8c4 Dmitry Safonov 2022-08-18 172 { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 173 unsigned int i; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 174 int err; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 175 f4c3873630fc8c4 Dmitry Safonov 2022-08-18 176 /* slow-path */ f4c3873630fc8c4 Dmitry Safonov 2022-08-18 177 mutex_lock(&cpool_mutex); f4c3873630fc8c4 Dmitry Safonov 2022-08-18 178 for (i = 0; i < last_allocated; i++) { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 179 if (cpool[i].alg && !strcmp(cpool[i].alg, alg)) { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 180 if (kref_read(&cpool[i].kref) > 0) { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 181 kref_get(&cpool[i].kref); f4c3873630fc8c4 Dmitry Safonov 2022-08-18 182 goto out; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 183 } else { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 184 break; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 185 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 186 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 187 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 188 f4c3873630fc8c4 Dmitry Safonov 2022-08-18 189 for (i = 0; i < last_allocated; i++) { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 190 if (!cpool[i].alg) f4c3873630fc8c4 Dmitry Safonov 2022-08-18 191 break; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 192 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 193 if (i >= CPOOL_SIZE) { f4c3873630fc8c4 Dmitry Safonov 2022-08-18 194 err = -ENOSPC; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 195 goto out; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 196 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 197 f4c3873630fc8c4 Dmitry Safonov 2022-08-18 198 err = __cpool_alloc_ahash(&cpool[i], alg); f4c3873630fc8c4 Dmitry Safonov 2022-08-18 199 if (!err && last_allocated <= i) f4c3873630fc8c4 Dmitry Safonov 2022-08-18 200 last_allocated++; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 201 out: f4c3873630fc8c4 Dmitry Safonov 2022-08-18 202 mutex_unlock(&cpool_mutex); f4c3873630fc8c4 Dmitry Safonov 2022-08-18 @203 return err ?: (int)i; f4c3873630fc8c4 Dmitry Safonov 2022-08-18 204 } f4c3873630fc8c4 Dmitry Safonov 2022-08-18 205 EXPORT_SYMBOL_GPL(crypto_pool_alloc_ahash); f4c3873630fc8c4 Dmitry Safonov 2022-08-18 206 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
