gdbstub/gdbstub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
The later handler if conditionally compiled only for Linux but we
forgot to ensure we don't advertise it lest we confuse our BSD
brethren.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
gdbstub/gdbstub.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
index 2a66371aa5..0760d78685 100644
--- a/gdbstub/gdbstub.c
+++ b/gdbstub/gdbstub.c
@@ -1468,7 +1468,7 @@ static void handle_query_supported(GArray *params, void *user_ctx)
";ReverseStep+;ReverseContinue+");
}
-#ifdef CONFIG_USER_ONLY
+#if defined(CONFIG_USER_ONLY) && defined(CONFIG_LINUX)
if (gdbserver_state.c_cpu->opaque) {
g_string_append(gdbserver_state.str_buf, ";qXfer:auxv:read+");
}
--
2.39.2
On 3/4/23 14:02, Alex Bennée wrote: > The later handler if conditionally compiled only for Linux but we > forgot to ensure we don't advertise it lest we confuse our BSD > brethren. > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > gdbstub/gdbstub.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c > index 2a66371aa5..0760d78685 100644 > --- a/gdbstub/gdbstub.c > +++ b/gdbstub/gdbstub.c > @@ -1468,7 +1468,7 @@ static void handle_query_supported(GArray *params, void *user_ctx) > ";ReverseStep+;ReverseContinue+"); > } > > -#ifdef CONFIG_USER_ONLY > +#if defined(CONFIG_USER_ONLY) && defined(CONFIG_LINUX) > if (gdbserver_state.c_cpu->opaque) { > g_string_append(gdbserver_state.str_buf, ";qXfer:auxv:read+"); > } Fixes: 51c623b0de ("gdbstub: add support to Xfer:auxv:read: packet") Reported-by: Warner Losh <imp@bsdimp.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
On Mon, Apr 3, 2023, 6:37 AM Philippe Mathieu-Daudé <philmd@linaro.org> wrote: > On 3/4/23 14:02, Alex Bennée wrote: > > The later handler if conditionally compiled only for Linux but we > > forgot to ensure we don't advertise it lest we confuse our BSD > > brethren. > > > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > > --- > > gdbstub/gdbstub.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c > > index 2a66371aa5..0760d78685 100644 > > --- a/gdbstub/gdbstub.c > > +++ b/gdbstub/gdbstub.c > > @@ -1468,7 +1468,7 @@ static void handle_query_supported(GArray *params, > void *user_ctx) > > ";ReverseStep+;ReverseContinue+"); > > } > > > > -#ifdef CONFIG_USER_ONLY > > +#if defined(CONFIG_USER_ONLY) && defined(CONFIG_LINUX) > > if (gdbserver_state.c_cpu->opaque) { > > g_string_append(gdbserver_state.str_buf, ";qXfer:auxv:read+"); > > } > > Fixes: 51c623b0de ("gdbstub: add support to Xfer:auxv:read: packet") > Reported-by: Warner Losh <imp@bsdimp.com> > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> > Reviewed-by: Warner Losh <imp@bsdimp.com> Tested-by: Warner Losh <imp@bsdimp.com> I can debug with this now. Cross debugging of binaries in a chroot with gdb has issues though... that's orthogonal to this patch... Warner >
© 2016 - 2024 Red Hat, Inc.