JS: storage.virtualQuit() does not free the virtual reference

[Derek Jamison]

Describe the bug.

In my script: I use storage.virtualInit(image) and .virtualMount(). Write some content to "/mnt/bug.txt". Call storage.virtualQuit. When I call usbdisk.start(image) the app hangs; blocked on a resource since virtualQuit did not free the resource.

I think the end of js_storage_virtual_quit needs to do something like:

    if(storage->virtual) {
        storage_file_free(storage->virtual);
        storage->virtual = NULL;
    }
    mjs_return(mjs, MJS_UNDEFINED);

Reproduction

The following script will demonstrate that after a storage.virtualInit that storage.virtualQuit does not release resources, so usbdisk.start will block.

let usbdisk = require("usbdisk");
let storage = require("storage");

let image = __dirpath + "/bug.img";

print("Checking for Image...");
if (storage.exists(image)) {
  print ("Storage Exists. Removing.");
  storage.remove(image);
}

print ("Creating Storage...");
usbdisk.createImage(image, 8 * 1024 * 1024);

print ("Copying Payload...")
storage.virtualInit(image);
storage.virtualMount();
storage.write("/mnt/demo.txt", "Hello");
storage.virtualQuit();

print("Attaching storage...");
usbdisk.start(image);
// BUG: The `usbdisk.start` never returns.

print("Waiting for storage to detatch...");
while (!usbdisk.wasEjected()) {
    delay(1000);
}
print("Stopping storage...");
usbdisk.stop();
print("Done.");

Target

js

Logs

No response

Anything else?

I confirmed that adding the storage_file_free call in js_storage_virtual_quit makes the JavaScript works as expected.

[WillyJL]

great catch! will have it fixed asap

[WillyJL]

fixed via 2f06a01, thank you!

[Derek Jamison]

It looks like the code got added to js_storage_virtual_mount instead of js_storage_virtual_quit.

[WillyJL]

Oh my god hahahahha

I need more sleep, I'm so sorry this was my bad

[WillyJL]

Fix was released in mntm-002