1096591 - Hang and reboot of Android system with this video

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: Audio/Video, defect)

Description

[Martijn Wargers (dead)]

Using current trunk build on Nexus 5, I get a hang of the system and then a reboot when visiting http://people.mozilla.org/~mwargers/tests/videos/lime10s_red10s.mp4

Comment 1

[Aaron Train [:aaronmt]]

Same device same build, I don't get any hang or crash. The video player reports a time-code of :20, however it plays and stops instantly for me.

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

(In reply to Aaron Train [:aaronmt] from comment #1)
> Same device same build, I don't get any hang or crash. The video player
> reports a time-code of :20, however it plays and stops instantly for me.

Same. I think this video has really like 4 frames, so maybe we have a problem with this super long frame time.

Comment 3

[James Willcox (:snorp) ([e-mail]) (he/him)]

OK, pretty sure I know what's going on here. We queue the 4 frames as in put, and then the MP4Reader calls Drain() because it's all done. We don't implement that properly, since we treat it as a Flush() and throw away all the pending output frames.

Comment 5

[Martijn Wargers (dead)]

I filed bug 1100116 on it, because it's also crashing on FirefoxOS in the browser.

Comment 6

[ykolomiyets]

This specific video plays fine for me. But i have the same issue randomly with other mp4 videos that would totally crash my phone. This only happens when using built-in video player.

This is in Marshmallow on Nexus 5X. Marshmallow on Nexus 5 didn't have this problem.

Comment 7

[Phillip Sz]

I have this problem with for me random mp4 videos with Android 6.0 in nexus 5 hammerhead

Comment 8

[Phillip Sz]

but I think thats https://bugzilla.mozilla.org/show_bug.cgi?id=1218949 sorry :)

Comment 9

[John Lin [:jhlin][:jolin]]

Cannot reproduce on Nexus 5/Android 4.4 with 57 nightly.

The problem described in comment 3 no longer exists in recent releases (all decoded frames will be returned for Drain()). Close this as fixed for now.