mirror of https://gitlab.com/qemu-project/qemu
You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
81143b7f9d
Currently, execlog searches for a space separator between the instruction mnemonic and operands, but some disassemblers, e.g. Alpha's, use a tab separator instead; this results in a null pointer being passed as the haystack in g_strstr during a subsequent register search, i.e. undefined behavior, because of a missing null check. This patch adds tab to the separator search and a null check on the result. Also, an affected pointer is changed to const. Lastly, a break statement was added to immediately terminate the register search when a user-requested register is found in the current instruction as a trivial optimization, because searching for the remaining requested registers is unnecessary once one is found. Suggested-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Yodel Eldar <yodel.eldar@gmail.com> Message-ID: <20250630164124.26315-2-yodel.eldar@gmail.com> Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-ID: <20250710104531.3099313-4-alex.bennee@linaro.org> |
6 days ago | |
|---|---|---|
| .. | ||
| bbv.c | 7 months ago | |
| cache.c | 3 months ago | |
| cflow.c | 3 months ago | |
| drcov.c | 2 years ago | |
| execlog.c | 6 days ago | |
| hotblocks.c | 3 months ago | |
| hotpages.c | 3 months ago | |
| howvec.c | 3 months ago | |
| hwprofile.c | 3 months ago | |
| ips.c | 1 month ago | |
| lockstep.c | 12 months ago | |
| meson.build | 1 month ago | |
| stoptrigger.c | 6 months ago | |
| win32_linker.c | 2 years ago | |