mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 15:22:21 -04:00
perf annotate: Return errors from disasm_line__parse_powerpc()
In disasm_line__parse_powerpc() , return code from function disasm_line__parse() is ignored. This will result in bad results if the disasm_line__parse() fails to disasm the line. Use the return code to fix this. Signed-off-by: Athira Rajeev <atrajeev@linux.ibm.com> Tested-By: Venkat Rao Bagalkote <venkat88@linux.ibm.com> Link: https://lore.kernel.org/r/20250304154114.62093-2-atrajeev@linux.ibm.com Signed-off-by: Namhyung Kim <namhyung@kernel.org>
This commit is contained in:
committed by
Namhyung Kim
parent
dab8c32ece
commit
4c3f09e35c
@@ -976,6 +976,7 @@ static int disasm_line__parse_powerpc(struct disasm_line *dl, struct annotate_ar
|
||||
char *tmp_raw_insn, *name_raw_insn = skip_spaces(line);
|
||||
char *name = skip_spaces(name_raw_insn + RAW_BYTES);
|
||||
int disasm = 0;
|
||||
int ret = 0;
|
||||
|
||||
if (args->options->disassembler_used)
|
||||
disasm = 1;
|
||||
@@ -984,7 +985,7 @@ static int disasm_line__parse_powerpc(struct disasm_line *dl, struct annotate_ar
|
||||
return -1;
|
||||
|
||||
if (disasm)
|
||||
disasm_line__parse(name, namep, rawp);
|
||||
ret = disasm_line__parse(name, namep, rawp);
|
||||
else
|
||||
*namep = "";
|
||||
|
||||
@@ -998,7 +999,7 @@ static int disasm_line__parse_powerpc(struct disasm_line *dl, struct annotate_ar
|
||||
if (disasm)
|
||||
dl->raw.raw_insn = be32_to_cpu(dl->raw.raw_insn);
|
||||
|
||||
return 0;
|
||||
return ret;
|
||||
}
|
||||
|
||||
static void annotation_line__init(struct annotation_line *al,
|
||||
|
||||
Reference in New Issue
Block a user