<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Dec 9, 2024 at 8:07 PM Tom Tromey <<a href="mailto:tromey@adacore.com">tromey@adacore.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I needed to use gdbreplay today.  It didn't work quite right, and<br>
using "set debug remote 1" showed that gdb was rejecting some<br>
responses like:<br>
<br>
  [remote] Sending packet: $vCont?#49<br>
  [remote] Junk: #vCont?<br>
  [remote] Junk: 8vCont?<br>
  [remote] Junk: 3vCont?<br>
  [remote] Received Ack<br>
<br>
The checksum recalculation seems to have gone wrong.  Looking at the<br>
code, it seems like 'where_csum' is calculated inconsistently: in the<br>
main loop it is after the '#' but in the "== 0" case it is before the<br>
'#'.<br>
<br>
This patch fixes the problem and also avoids a string copy.<br>
<br>
CC: Alexandra Hájková <<a href="mailto:ahajkova@redhat.com" target="_blank">ahajkova@redhat.com</a>><br>
---<br><br></blockquote><div><br></div><div>Hi Tom,</div><div><br></div><div>thank you for fixing this mistake of mine.</div><div><br></div><div> <br></div></div></div>