memory access out of bounds about vmsg

[Kagami]

Thanks for report, David. I will check it.

[Kagami]

Hi, @oleg-preply. It's ok to bump TOTAL_MEMORY, you don't need to recompile wasm module for that. Basically WebAssembly.Memory is an array of bytes you create for your wasm module, so you can use any size it would need. Browsers might not allow to allocate really big e.g. 1GB memory arrays but 100-200MB should be fine even on mobile.

When I was creating this library I thought 16MB is enough for most MP3 records, but seems like it's not. We should either allow to setup TOTAL_MEMORY value via option passed to record/Recorder. Or send compressed MP3 samples from worker to main thread right away, as described in #23.

Second option is better because memory would be consumed in a more optimal way, without allocating MP3 twice, first at worker and then at main thread and spawning wasm module with a lot of memory. But first option is easier to implement and should be ok as short-term solution.

(Btw, it's also possible to grow memory on demand, but I haven't implemented it either.)

[RamboWasReal]

I searched online and I did not foud anything to help me. I'm not a profesionnal, so if you know where to look in the code to correct the situation please tell me and I will try to solve it on my own.

Thanks.

[0xAlwaysDumpling]

Anyone know how to solve this problem? I am trying to record audio up to 30 minutes.

[a-h-abid]

Hi @Kagami, have you managed to solve this issue? I'm also having trouble when trying to record more than 15mins.

[oleg-preply]

Hi! I've faced the same on the 8th minute.

The reason might be the memory limit set in the package to 16MB. The obvious way seems to try to increase it, but:

• changing the value requires also recompiling .wasm with a new value, as commented above the variable via the link ☝️, also like proposed here. So there is no way dynamically adjust that option since it requires compiling/downloading another .wasm module
• I'm not sure what are downsides of increasing size. Like in the solution above folks noticed worse performance.

In the same time, I've tried to increase only TOTAL_MEMORY to 50MB (but didn't recompile .wasm) and succeeded in the recording of 15minutes 🎉 So maybe .wasm module is already compiled with a bigger value?
Also, I'm wondering how it can affect mobile devices, where OS memory limits might be lower 🤔

@Kagami do you think the increase of TOTAL_MEMORY is a legit solution?

Thanks in advance!