diff for duplicates of <202201201534.RoB5fAbP-lkp@intel.com> diff --git a/a/1.txt b/N1/1.txt index 423e9f4..93a4a0f 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,15 +1,6 @@ -CC: kbuild-all(a)lists.01.org -CC: linux-kernel(a)vger.kernel.org -TO: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> -CC: Mark Brown <broonie@kernel.org> -CC: "Pierre-Louis Bossart" <pierre-louis.bossart@linux.intel.com> - tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1d1df41c5a33359a00e919d54eaebfb789711fdc commit: 36ad9bf1d93d66b901342eab9f8ed6c1537655a6 ASoC: qdsp6: audioreach: add topology support -date: 3 months ago -:::::: branch date: 22 hours ago -:::::: commit date: 3 months ago config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp(a)intel.com/config) compiler: arceb-elf-gcc (GCC) 11.2.0 @@ -33,7 +24,6 @@ sound/soc/qcom/qdsp6/topology.c:391 audioreach_parse_cont_tokens() error: potent vim +/sg +336 sound/soc/qcom/qdsp6/topology.c -36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 302 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 303 static struct audioreach_sub_graph *audioreach_parse_sg_tokens(struct q6apm *apm, 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 304 struct snd_soc_tplg_private *private) 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 305 { @@ -49,6 +39,21 @@ vim +/sg +336 sound/soc/qcom/qdsp6/topology.c 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 315 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 316 while (tkn_count <= (le32_to_cpu(sg_array->num_elems) - 1)) { 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 317 switch (le32_to_cpu(sg_elem->token)) { + +This code is very trustful of endian data. Probably that means it comes +from the firmware or something? People will file for CVEs when they +find that we're trusting firmware to be correct... + +Also I really wish that some day we will be able to turn on GCC's +unitialized variable checking again. There is no way I'm going to put +up with looking through these warnings for year after year. At least +fix the W=2 uninitialized variable warnings. + +sound/soc/qcom/qdsp6/topology.c:342:20: warning: ‘sg’ may be used uninitialized in this function [-Wmaybe-uninitialized] + sg->scenario_id = le32_to_cpu(sg_elem->value); +sound/soc/qcom/qdsp6/topology.c:310:31: note: ‘sg’ was declared here + struct audioreach_sub_graph *sg; + ^~ 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 318 case AR_TKN_U32_SUB_GRAPH_INSTANCE_ID: 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 319 sub_graph_id = le32_to_cpu(sg_elem->value); 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 320 sg = audioreach_tplg_alloc_sub_graph(apm, sub_graph_id, &found); @@ -135,7 +140,6 @@ vim +/sg +336 sound/soc/qcom/qdsp6/topology.c 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 401 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 @402 return cont; 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 403 } -36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 404 --- 0-DAY CI Kernel Test Service, Intel Corporation diff --git a/a/content_digest b/N1/content_digest index e634396..66f4e18 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,21 +1,12 @@ - "From\0kernel test robot <lkp@intel.com>\0" + "From\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Subject\0sound/soc/qcom/qdsp6/topology.c:336 audioreach_parse_sg_tokens() error: potentially dereferencing uninitialized 'sg'.\0" - "Date\0Thu, 20 Jan 2022 15:52:01 +0800\0" - "To\0kbuild@lists.01.org\0" + "Date\0Thu, 20 Jan 2022 11:19:11 +0300\0" + "To\0kbuild-all@lists.01.org\0" "\01:1\0" "b\0" - "CC: kbuild-all(a)lists.01.org\n" - "CC: linux-kernel(a)vger.kernel.org\n" - "TO: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>\n" - "CC: Mark Brown <broonie@kernel.org>\n" - "CC: \"Pierre-Louis Bossart\" <pierre-louis.bossart@linux.intel.com>\n" - "\n" "tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master\n" "head: 1d1df41c5a33359a00e919d54eaebfb789711fdc\n" "commit: 36ad9bf1d93d66b901342eab9f8ed6c1537655a6 ASoC: qdsp6: audioreach: add topology support\n" - "date: 3 months ago\n" - ":::::: branch date: 22 hours ago\n" - ":::::: commit date: 3 months ago\n" "config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp(a)intel.com/config)\n" "compiler: arceb-elf-gcc (GCC) 11.2.0\n" "\n" @@ -39,7 +30,6 @@ "\n" "vim +/sg +336 sound/soc/qcom/qdsp6/topology.c\n" "\n" - "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 302 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 303 static struct audioreach_sub_graph *audioreach_parse_sg_tokens(struct q6apm *apm,\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 304 \t\t\t\t\t\t struct snd_soc_tplg_private *private)\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 305 {\n" @@ -55,6 +45,21 @@ "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 315 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 316 \twhile (tkn_count <= (le32_to_cpu(sg_array->num_elems) - 1)) {\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 317 \t\tswitch (le32_to_cpu(sg_elem->token)) {\n" + "\n" + "This code is very trustful of endian data. Probably that means it comes\n" + "from the firmware or something? People will file for CVEs when they\n" + "find that we're trusting firmware to be correct...\n" + "\n" + "Also I really wish that some day we will be able to turn on GCC's\n" + "unitialized variable checking again. There is no way I'm going to put\n" + "up with looking through these warnings for year after year. At least\n" + "fix the W=2 uninitialized variable warnings.\n" + "\n" + "sound/soc/qcom/qdsp6/topology.c:342:20: warning: \342\200\230sg\342\200\231 may be used uninitialized in this function [-Wmaybe-uninitialized]\n" + " sg->scenario_id = le32_to_cpu(sg_elem->value);\n" + "sound/soc/qcom/qdsp6/topology.c:310:31: note: \342\200\230sg\342\200\231 was declared here\n" + " struct audioreach_sub_graph *sg;\n" + " ^~\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 318 \t\tcase AR_TKN_U32_SUB_GRAPH_INSTANCE_ID:\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 319 \t\t\tsub_graph_id = le32_to_cpu(sg_elem->value);\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 320 \t\t\tsg = audioreach_tplg_alloc_sub_graph(apm, sub_graph_id, &found);\n" @@ -141,10 +146,9 @@ "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 401 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 @402 \treturn cont;\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 403 }\n" - "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 404 \n" "\n" "---\n" "0-DAY CI Kernel Test Service, Intel Corporation\n" https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org -4a3ade853de0634848fd382e27cf8ba2aae0bde8c1d3aa95a4251b9e817e61a7 +e5e0a607d826a9da65d17bf80c99b99b26ed480f8cb541ded2eba9b10675bdf5
diff --git a/a/1.txt b/N2/1.txt index 423e9f4..9efdf45 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,16 +1,7 @@ -CC: kbuild-all(a)lists.01.org -CC: linux-kernel(a)vger.kernel.org -TO: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> -CC: Mark Brown <broonie@kernel.org> -CC: "Pierre-Louis Bossart" <pierre-louis.bossart@linux.intel.com> - tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1d1df41c5a33359a00e919d54eaebfb789711fdc commit: 36ad9bf1d93d66b901342eab9f8ed6c1537655a6 ASoC: qdsp6: audioreach: add topology support -date: 3 months ago -:::::: branch date: 22 hours ago -:::::: commit date: 3 months ago -config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp(a)intel.com/config) +config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp@intel.com/config) compiler: arceb-elf-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate @@ -33,7 +24,6 @@ sound/soc/qcom/qdsp6/topology.c:391 audioreach_parse_cont_tokens() error: potent vim +/sg +336 sound/soc/qcom/qdsp6/topology.c -36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 302 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 303 static struct audioreach_sub_graph *audioreach_parse_sg_tokens(struct q6apm *apm, 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 304 struct snd_soc_tplg_private *private) 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 305 { @@ -49,6 +39,21 @@ vim +/sg +336 sound/soc/qcom/qdsp6/topology.c 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 315 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 316 while (tkn_count <= (le32_to_cpu(sg_array->num_elems) - 1)) { 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 317 switch (le32_to_cpu(sg_elem->token)) { + +This code is very trustful of endian data. Probably that means it comes +from the firmware or something? People will file for CVEs when they +find that we're trusting firmware to be correct... + +Also I really wish that some day we will be able to turn on GCC's +unitialized variable checking again. There is no way I'm going to put +up with looking through these warnings for year after year. At least +fix the W=2 uninitialized variable warnings. + +sound/soc/qcom/qdsp6/topology.c:342:20: warning: ‘sg’ may be used uninitialized in this function [-Wmaybe-uninitialized] + sg->scenario_id = le32_to_cpu(sg_elem->value); +sound/soc/qcom/qdsp6/topology.c:310:31: note: ‘sg’ was declared here + struct audioreach_sub_graph *sg; + ^~ 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 318 case AR_TKN_U32_SUB_GRAPH_INSTANCE_ID: 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 319 sub_graph_id = le32_to_cpu(sg_elem->value); 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 320 sg = audioreach_tplg_alloc_sub_graph(apm, sub_graph_id, &found); @@ -135,8 +140,7 @@ vim +/sg +336 sound/soc/qcom/qdsp6/topology.c 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 401 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 @402 return cont; 36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 403 } -36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 404 --- 0-DAY CI Kernel Test Service, Intel Corporation -https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org +https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org diff --git a/a/content_digest b/N2/content_digest index e634396..d0b9a94 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,22 +1,19 @@ - "From\0kernel test robot <lkp@intel.com>\0" + "From\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Subject\0sound/soc/qcom/qdsp6/topology.c:336 audioreach_parse_sg_tokens() error: potentially dereferencing uninitialized 'sg'.\0" - "Date\0Thu, 20 Jan 2022 15:52:01 +0800\0" - "To\0kbuild@lists.01.org\0" - "\01:1\0" + "Date\0Thu, 20 Jan 2022 11:19:11 +0300\0" + "To\0kbuild@lists.01.org" + " Srinivas Kandagatla <srinivas.kandagatla@linaro.org>\0" + "Cc\0lkp@intel.com" + kbuild-all@lists.01.org + linux-kernel@vger.kernel.org + Mark Brown <broonie@kernel.org> + " Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>\0" + "\00:1\0" "b\0" - "CC: kbuild-all(a)lists.01.org\n" - "CC: linux-kernel(a)vger.kernel.org\n" - "TO: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>\n" - "CC: Mark Brown <broonie@kernel.org>\n" - "CC: \"Pierre-Louis Bossart\" <pierre-louis.bossart@linux.intel.com>\n" - "\n" "tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master\n" "head: 1d1df41c5a33359a00e919d54eaebfb789711fdc\n" "commit: 36ad9bf1d93d66b901342eab9f8ed6c1537655a6 ASoC: qdsp6: audioreach: add topology support\n" - "date: 3 months ago\n" - ":::::: branch date: 22 hours ago\n" - ":::::: commit date: 3 months ago\n" - "config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp(a)intel.com/config)\n" + "config: arc-randconfig-m031-20220117 (https://download.01.org/0day-ci/archive/20220120/202201201534.RoB5fAbP-lkp@intel.com/config)\n" "compiler: arceb-elf-gcc (GCC) 11.2.0\n" "\n" "If you fix the issue, kindly add following tag as appropriate\n" @@ -39,7 +36,6 @@ "\n" "vim +/sg +336 sound/soc/qcom/qdsp6/topology.c\n" "\n" - "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 302 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 303 static struct audioreach_sub_graph *audioreach_parse_sg_tokens(struct q6apm *apm,\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 304 \t\t\t\t\t\t struct snd_soc_tplg_private *private)\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 305 {\n" @@ -55,6 +51,21 @@ "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 315 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 316 \twhile (tkn_count <= (le32_to_cpu(sg_array->num_elems) - 1)) {\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 317 \t\tswitch (le32_to_cpu(sg_elem->token)) {\n" + "\n" + "This code is very trustful of endian data. Probably that means it comes\n" + "from the firmware or something? People will file for CVEs when they\n" + "find that we're trusting firmware to be correct...\n" + "\n" + "Also I really wish that some day we will be able to turn on GCC's\n" + "unitialized variable checking again. There is no way I'm going to put\n" + "up with looking through these warnings for year after year. At least\n" + "fix the W=2 uninitialized variable warnings.\n" + "\n" + "sound/soc/qcom/qdsp6/topology.c:342:20: warning: \342\200\230sg\342\200\231 may be used uninitialized in this function [-Wmaybe-uninitialized]\n" + " sg->scenario_id = le32_to_cpu(sg_elem->value);\n" + "sound/soc/qcom/qdsp6/topology.c:310:31: note: \342\200\230sg\342\200\231 was declared here\n" + " struct audioreach_sub_graph *sg;\n" + " ^~\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 318 \t\tcase AR_TKN_U32_SUB_GRAPH_INSTANCE_ID:\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 319 \t\t\tsub_graph_id = le32_to_cpu(sg_elem->value);\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 320 \t\t\tsg = audioreach_tplg_alloc_sub_graph(apm, sub_graph_id, &found);\n" @@ -141,10 +152,9 @@ "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 401 \n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 @402 \treturn cont;\n" "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 403 }\n" - "36ad9bf1d93d66 Srinivas Kandagatla 2021-10-26 404 \n" "\n" "---\n" "0-DAY CI Kernel Test Service, Intel Corporation\n" - https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org + https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org -4a3ade853de0634848fd382e27cf8ba2aae0bde8c1d3aa95a4251b9e817e61a7 +39d443cb5d8e07101c32f296fb00d2ea6eeadd0f816be040cf478cd137e21ce6
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.