xen/misra: xen-analysis.py: Fix latent bug

Currenly there is a latent bug that is not triggered because
the function cppcheck_merge_txt_fragments is called with the
parameter strip_paths having a list of only one element.

The bug is that the split function should not be in the
loop for strip_paths, but one level before, fix it.

Fixes: 02b26c02c7 ("xen/scripts: add cppcheck tool to the xen-analysis.py script")
Signed-off-by: Luca Fancellu <luca.fancellu@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
Tested-by: Stefano Stabellini <sstabellini@kernel.org>
This commit is contained in:
Luca Fancellu authored and Jan Beulich committed 2023-05-30 11:57:02 +02:00
1 parent e56f2106b6
commit 8bd504290b
1 file changed
+2 -2
@@ -104,8 +104,8 @@ def cppcheck_merge_txt_fragments(fragments_list, out_txt_file, strip_paths):
for path in strip_paths:
text_report_content[i] = text_report_content[i].replace(
path + "/", "")
# Split by : separator
text_report_content[i] = text_report_content[i].split(":")
# Split by : separator
text_report_content[i] = text_report_content[i].split(":")
# sort alphabetically for second field (misra rule) and as second
# criteria for the first field (file name)
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.