Fix load for transfer operation

Author: tolauwae

src/Debug/debugger.cpp

@@ -1125,18 +1125,18 @@ void Debugger::freeState(Module *m, uint8_t *interruptData) {
     debug("done with first msg\n");
 }
 
-void load(uint8_t *bytes, Module* m) {
-    auto start = read_B32(&bytes);
-    auto limit = read_B32(&bytes);
+void Debugger::load(uint8_t *bytes, Module* m) {
+    auto start = read_B8(&bytes);
+    auto limit = read_B8(&bytes);
     auto total_bytes = limit - start + 1;
+    this->channel->write("loading into %u - %u \n", start, limit);
     memcpy(m->memory.bytes + start, bytes, total_bytes);
 }
 
 void Debugger::transfer(Module *m, uint8_t *interruptData) {
-    uint8_t *cursor = nullptr;
+    uint8_t *cursor = interruptData;
     uint8_t *end = nullptr;
-    cursor = interruptData + 1;  // skip interruptLoadSnapshot
-    uint32_t len = read_B32(&cursor);
+    uint16_t len = read_B16(&cursor);
     end = cursor + len;
 
     while (cursor < end) {

src/Debug/debugger.h

@@ -211,6 +211,8 @@ class Debugger {
 
     bool saveState(Module *m, uint8_t *interruptData);
 
+    void load(uint8_t *bytes, Module* m);
+
     void transfer(Module *m, uint8_t *interruptData);
 
     static uintptr_t readPointer(uint8_t **data);

src/Utils/util.cpp

@@ -347,6 +347,13 @@ uint16_t read_B16(uint8_t **bytes) {
     return n;
 }
 
+uint8_t read_B8(uint8_t **bytes) {
+    uint8_t *b = *bytes;
+    uint8_t n = b[0];
+    *bytes += 1;
+    return n;
+}
+
 int read_B32_signed(uint8_t **bytes) {
     uint8_t *b = *bytes;
     int n = (b[0] << 24) + (b[1] << 16) + (b[2] << 8) + b[3];

src/Utils/util.h

@@ -121,6 +121,7 @@ double wa_fmin(double a, double b);
 // legacy util functions (todo remove)
 uint32_t read_B32(uint8_t **bytes);
 uint16_t read_B16(uint8_t **bytes);
+uint8_t read_B8(uint8_t **bytes);
 int read_B32_signed(uint8_t **bytes);
 uint32_t read_L32(uint8_t **bytes);
 void chars_as_hexa(unsigned char *dest, unsigned char *source,