Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

memory access out of bounds #27

Closed
hjl07259 opened this issue Jun 28, 2020 · 10 comments
Closed

memory access out of bounds #27

hjl07259 opened this issue Jun 28, 2020 · 10 comments

Comments

@hjl07259
Copy link

error:memory access out of bounds。Excuse me, how should I solve this problem?

@chanhdatng
Copy link

did you solve this problem? @hjl07259

@cycjimmy
Copy link
Owner

cycjimmy commented Nov 3, 2020

This problem may be related to the encoding of video or the setting of videoBufferSize. See the issue in jsmpeg for more information. phoboslab/jsmpeg#268

@felipecrs
Copy link

felipecrs commented Mar 10, 2022

@cycjimmy is there a chance the code from this repo is missing emscripten-core/emscripten#5289? I'm trying to debug this issue. Haven't found anything useful yet, I just know that it throws at this line in Javascript:

this.bytesWritten += this.bufferWrite(buffers);

And then at the C code at:

https://github.com/phoboslab/jsmpeg/blob/b5799bfc213c7732e1f68754d9244f47258d8be5/src/wasm/mpeg1.c#L778

mpeg1_decoder_t *self = malloc(sizeof(mpeg1_decoder_t));

@cycjimmy
Copy link
Owner

jsmpeg-player is based on jsmpeg. It is recommended to submit this issue in the jsmpeg repo. @felipecrs

@felipecrs
Copy link

Yes, but there is no indication of which revision of jsmpeg is used here. Is it up to date with the latest jsmpeg upstream?

@cycjimmy
Copy link
Owner

I have just updated to the latest jsmpeg.

@felipecrs
Copy link

@cycjimmy that's awesome! And I wasn't able to reproduce the issue since the update! But let me try it a bit more before concluding anything.

@felipecrs
Copy link

Ok, this issue happened a lot before. Since the upgrade, I never observed it again. I'm okay to say that it fixed the issue for me.

Thank you!

@VuIt96
Copy link

VuIt96 commented Feb 1, 2023

I updated to the latest version but still have the same problem
Capture

@vipulmore01
Copy link

vipulmore01 commented Feb 23, 2024

@VuIt96 did you fixed that issue ? , I am also getting the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants