Menu

#101 Patch for remote monitor binary memdump command decoding error

v2.4.x
closed-accepted
None
bugfix
2014-11-25
2014-11-25
No

Vice remote monitor binary memdump feature does not work - it decodes the command wrong.

The attached patch fixes this problem.

Example (test case) - this command dumps the kernal rom - it will work when the patch is applied:
printf "\x02\x05\x01\x00\xe0\xff\xff\x00" | nc localhost 6510 > /tmp/kernal.dmp

For information on the remote monitor binary command format please look at src/monitor/monitor_network.c around line 180-230

Patch attached

/Uffe

1 Attachments

Discussion

  • Marco van den Heuvel

    Applied in rev 28792, thanks for the patch.

     
  • Marco van den Heuvel

    • status: open --> closed-accepted
    • assigned_to: Marco van den Heuvel
     

Log in to post a comment.