gdb: fix possible nullptr dereference in a remote_debug_printf call
authorAndrew Burgess <aburgess@redhat.com>
Tue, 18 Jul 2023 12:52:20 +0000 (13:52 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Thu, 3 Aug 2023 08:54:27 +0000 (09:54 +0100)
commit1720b64f735ff2798ab50ea9e2a40ab42af6cc6e
tree52dcaa428251885ded91eaa0dcf1068a028242a8
parente34acfce343d53290e72c317213a28327bc52fb0
gdb: fix possible nullptr dereference in a remote_debug_printf call

While working on another patch I triggered a segfault from within the
function remote_target::discard_pending_stop_replies.  Turns out this
was caused by a cut&paste error introduced in this commit:

  commit df5ad102009c41ab4dfadbb8cfb8c8b2a02a4f78
  Date:   Wed Dec 1 09:40:03 2021 -0500

      gdb, gdbserver: detach fork child when detaching from fork parent

This commit adds a remote_debug_printf call that was copied from
earlier in the function, however, the new call wasn't updated to use
the appropriate local variable.  The local variable that it is using
might be nullptr, in which case we trigger undefined behaviour, and
could crash, which is what I was seeing.

Fixed by updating to use the correct local variable.
gdb/remote.c