PCB 1.3 added

README.md

@@ -83,7 +83,7 @@ The polling of BOOTSEL was creating some issues to the *BADUSB* detection so thi
 
 <img src="https://github.com/cecio/USBvalve/blob/main/pictures/reset_button.jpg" width="35%" height="35%" />
 
-~~In `PCB` you'll find also a version `1.3` of the board, with some holes on pads 1 and 3 to facilitate the mount of the button. But as you can see, you can also use your old version~~. (still waiting the new printed PCB to test it before publishing)
+In `PCB` you'll find also a version `1.3` of the board, with some holes on pads 1 and 3 to facilitate the mount of the button. But as you can see, you can also use your old version.
 
 If you are not using the *BADUSB* functions or if you prefer to have less coverage on detection but keep BOOTSEL usage, I'm also providing a firmware created with *bootsel* enabled (see folder and releases).
 

src/usb_device.c

@@ -73,14 +73,17 @@ int32_t tud_msc_read10_cb(uint8_t lun, uint32_t lba, uint32_t offset,
         flag_autorun = true;
     }
 
-    // Protect memory: only read from real disk blocks
+    // Return data from ramdisk, or zeros for blocks beyond it.
-    if (lba < DISK_BLOCK_NUM - 1) {
+    // The disk reports FAKE_DISK_BLOCK_NUM sectors but only
-        uint8_t const* addr = msc_disk[lba];
+    // DISK_BLOCK_NUM exist in RAM — the rest must read as empty.
-        memcpy(buffer, addr, bufsize);
+    if (lba < DISK_BLOCK_NUM) {
+        memcpy(buffer, msc_disk[lba], bufsize);
+    } else {
+        memset(buffer, 0, bufsize);
     }
 
     serial_printf("Read LBA: %u   Size: %u\r\n", (unsigned)lba, (unsigned)bufsize);
-    if (lba < DISK_BLOCK_NUM - 1) {
+    if (lba < DISK_BLOCK_NUM) {
         hex_dump(msc_disk[lba], MAX_DUMP_BYTES);
     }
 
@@ -105,14 +108,13 @@ int32_t tud_msc_write10_cb(uint8_t lun, uint32_t lba, uint32_t offset,
         flag_written = true;
     }
 
-    // Protect memory: only write to real disk blocks
+    // Only write to real disk blocks; silently discard beyond ramdisk
-    if (lba < DISK_BLOCK_NUM - 1) {
+    if (lba < DISK_BLOCK_NUM) {
-        uint8_t* addr = msc_disk[lba];
+        memcpy(msc_disk[lba], buffer, bufsize);
-        memcpy(addr, buffer, bufsize);
     }
 
     serial_printf("Write LBA: %u   Size: %u\r\n", (unsigned)lba, (unsigned)bufsize);
-    if (lba < DISK_BLOCK_NUM - 1) {
+    if (lba < DISK_BLOCK_NUM) {
         hex_dump(msc_disk[lba], MAX_DUMP_BYTES);
     }