Fix: BPS patch TARGET_COPY reading

src/types/patches/bpsPatch.ts

@@ -112,11 +112,18 @@ export default class BPSPatch extends Patch {
         sourceRelativeOffset += (offset & 1 ? -1 : +1) * (offset >> 1);
         await targetFile.write(await sourceFile.readAt(sourceRelativeOffset, length));
         sourceRelativeOffset += length;
-      } else {
+      } else if (action === BPSAction.TARGET_COPY) {
         const offset = await Patch.readUpsUint(patchFile);
         targetRelativeOffset += (offset & 1 ? -1 : +1) * (offset >> 1);
-        await targetFile.write(await targetFile.readAt(targetRelativeOffset, length));
-        targetRelativeOffset += length;
+        // WARN: you explicitly can't read the target file all at once, you have to read byte by
+        // byte, because later iterations of the loop may need to read data that was changed by
+        // earlier iterations of the loop.
+        for (let i = 0; i < length; i += 1) {
+          await targetFile.write(await targetFile.readAt(targetRelativeOffset, 1));
+          targetRelativeOffset += 1;
+        }
+      } else {
+        throw new Error(`BPS action ${action} isn't supported`);
       }
     }
   }