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

Regression: crash when using the same Transformer hundreds of times #2036

Open
1 task done
samsonjs opened this issue Jan 14, 2025 · 1 comment
Open
1 task done

Comments

@samsonjs
Copy link

Version

Media3 1.5.1

More version details

This first showed up in 1.4.1 and I bisected that and 1.4.0 to find this commit that introduced the crash: cd2a36f

Devices that reproduce the issue

  • Galaxy S22 / Android 14
  • Pixel 3 / Android 12
  • OnePlus 7 Pro / Android 11

Devices that do not reproduce the issue

None found

Reproducible in the demo app?

Not tested

Reproduction steps

Open this project and tap the Start Clipping button. You can watch logcat to see how many videos it processes before crashing, and it's usually by the 100 mark for me so I included 120 videos which should be enough to reproduce it.

https://drive.google.com/file/d/17riddvXjUakjOfoRcH2CtiAgxik-uBDt/view?usp=drive_link

Expected result

It should not crash

Actual result

It crashes

Media

See project here https://drive.google.com/file/d/17riddvXjUakjOfoRcH2CtiAgxik-uBDt/view?usp=drive_link

Bug Report

@samsonjs
Copy link
Author

samsonjs commented Jan 14, 2025

Bug report emailed, let me know if I can provide anything else to help nail this one down.

Also we have a work-around by just creating a new Transformer instance for every video, and that's totally fine as far as I can tell. Maybe we shouldn't be re-using the same instance like this in the first place? It used to work though.

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

No branches or pull requests

3 participants