[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 07/11] gdbstub: Use g_markup_printf_escaped()
From: |
Akihiko Odaki |
Subject: |
[PATCH v2 07/11] gdbstub: Use g_markup_printf_escaped() |
Date: |
Tue, 12 Sep 2023 15:58:02 +0900 |
g_markup_printf_escaped() is a safer alternative to simple printf() as
it automatically escapes values.
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
---
gdbstub/gdbstub.c | 36 +++++++++++++++++++++---------------
1 file changed, 21 insertions(+), 15 deletions(-)
diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
index 9db4af41c1..a4f2bf3723 100644
--- a/gdbstub/gdbstub.c
+++ b/gdbstub/gdbstub.c
@@ -373,28 +373,34 @@ static const char *get_feature_xml(const char *p, const
char **newp,
if (strncmp(p, "target.xml", len) == 0) {
if (!process->target_xml) {
GDBRegisterState *r;
- GString *xml = g_string_new("<?xml version=\"1.0\"?>");
+ g_autoptr(GPtrArray) xml = g_ptr_array_new_with_free_func(g_free);
- g_string_append(xml,
- "<!DOCTYPE target SYSTEM \"gdb-target.dtd\">"
- "<target>");
+ g_ptr_array_add(
+ xml,
+ g_strdup("<?xml version=\"1.0\"?>"
+ "<!DOCTYPE target SYSTEM \"gdb-target.dtd\">"
+ "<target>"));
if (cc->gdb_arch_name) {
- g_string_append_printf(xml,
- "<architecture>%s</architecture>",
- cc->gdb_arch_name(cpu));
+ g_ptr_array_add(
+ xml,
+ g_markup_printf_escaped("<architecture>%s</architecture>",
+ cc->gdb_arch_name(cpu)));
}
- g_string_append(xml, "<xi:include href=\"");
- g_string_append(xml, cc->gdb_core_xml_file);
- g_string_append(xml, "\"/>");
+ g_ptr_array_add(
+ xml,
+ g_markup_printf_escaped("<xi:include href=\"%s\"/>",
+ cc->gdb_core_xml_file));
for (r = cpu->gdb_regs; r; r = r->next) {
- g_string_append(xml, "<xi:include href=\"");
- g_string_append(xml, r->xml);
- g_string_append(xml, "\"/>");
+ g_ptr_array_add(
+ xml,
+ g_markup_printf_escaped("<xi:include href=\"%s\"/>",
+ r->xml));
}
- g_string_append(xml, "</target>");
+ g_ptr_array_add(xml, g_strdup("</target>"));
+ g_ptr_array_add(xml, NULL);
- process->target_xml = g_string_free(xml, false);
+ process->target_xml = g_strjoinv(NULL, (void *)xml->pdata);
}
return process->target_xml;
}
--
2.42.0
- [PATCH v2 00/11] gdbstub and TCG plugin improvements, Akihiko Odaki, 2023/09/12
- [PATCH v2 02/11] gdbstub: Fix target.xml response, Akihiko Odaki, 2023/09/12
- [PATCH v2 01/11] gdbstub: Fix target_xml initialization, Akihiko Odaki, 2023/09/12
- [PATCH v2 03/11] contrib/plugins: Use GRWLock in execlog, Akihiko Odaki, 2023/09/12
- [PATCH v2 04/11] gdbstub: Introduce GDBFeature structure, Akihiko Odaki, 2023/09/12
- [PATCH v2 05/11] target/arm: Move the reference to arm-core.xml, Akihiko Odaki, 2023/09/12
- [PATCH v2 06/11] hw/core/cpu: Return static value with gdb_arch_name(), Akihiko Odaki, 2023/09/12
- [PATCH v2 10/11] gdbstub: Remove gdb_has_xml variable, Akihiko Odaki, 2023/09/12
- [PATCH v2 09/11] target/ppc: Remove references to gdb_has_xml, Akihiko Odaki, 2023/09/12
- [PATCH v2 11/11] gdbstub: Replace gdb_regs with an array, Akihiko Odaki, 2023/09/12
- [PATCH v2 07/11] gdbstub: Use g_markup_printf_escaped(),
Akihiko Odaki <=
- [PATCH v2 08/11] target/arm: Remove references to gdb_has_xml, Akihiko Odaki, 2023/09/12