Closed
Bug 951117
Opened 11 years ago
Closed 11 years ago
[RTSP][Video][V1.3] Crash happened while stop playing RTSP streaming.
Categories
(Firefox OS Graveyard :: Gaia::Video, defect)
Tracking
(blocking-b2g:1.3+, firefox27 wontfix, firefox28 fixed, firefox29 fixed, b2g-v1.3 fixed)
People
(Reporter: whsu, Assigned: ethan)
Details
(Whiteboard: [FT:RIL])
Attachments
(5 files, 1 obsolete file)
* Description:
This problem happened on v1.3 build.
While FxOS is playing RTSP streaming the you try to tap "<" icon to go to previous page, the video app will crash.
Attach the following document,
- Video (WP_20131217_001.mp4)
- log & Crash Report (RTSP_Crash_Log_20131217.tar.gz)
* Reproduction steps:
1. Launch the following web page via browser
- http://goo.gl/lE2eE3
2. Tap the "Video test page"
3. After FxOS launches the video to play RTSP stream, please wait for 20 seconds.
4. Tap "<" icon to go to previous page
* Expected result:
FxOS goes to previous page without getting any problem
* Actual result:
A crash happened.
* Reproduction build: V1.3 (Buri)
- Gaia: 37142f72c422120401dbc90e1f5e8f689576bb8e
- Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/cc2ab73484b2
- BuildID 20131215004003
- Version 28.0a2
Reporter | ||
Comment 1•11 years ago
|
||
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → 1.3?
Assignee | ||
Comment 2•11 years ago
|
||
William, I can't extract your attachment.
$ tar zxvf RTSP_Crash_Log_20131217.tar.gz
tar: This does not look like a tar archive
tar: Skipping to next header
tar: Exiting with failure status due to previous errors
Reporter | ||
Comment 3•11 years ago
|
||
Reporter | ||
Comment 4•11 years ago
|
||
Hi, Ethan,
Thanks for the reminder.
You can use following command to extract it.
=> gzip -dc RTSP_Crash_Log_20131217.tar.gz | tar -zxf -
I also upload a zip (RTSP_Crash_Event_20131217.zip) version. FYI.
Updated•11 years ago
|
Whiteboard: [FT:RIL]
Comment 6•11 years ago
|
||
NI Sri for confirming blocking decision (per some recent email from Sri/Chris - video streaming is not a blocker for 1.3)
Flags: needinfo?(skasetti)
Assignee | ||
Comment 7•11 years ago
|
||
** Crash Point **
The crash happens here:
http://dxr.mozilla.org/mozilla-central/source/content/media/RtspMediaResource.cpp?from=RtspMediaResource.cpp#536
Program received signal SIGSEGV, Segmentation fault.
0x40ebda3c in mozilla::RtspMediaResource::OnDisconnected (this=0x440e0500, aTrackIdx=0 '\000',
aReason=2152398860) at /home/ethan/workspace/hg/mozilla-central/content/media/RtspMediaResource.cpp:536
536 mDecoder->NetworkError();
(gdb) bt
#0 0x40ebda3c in mozilla::RtspMediaResource::OnDisconnected (this=0x440e0500, aTrackIdx=0 '\000',
aReason=2152398860) at /home/ethan/workspace/hg/mozilla-central/content/media/RtspMediaResource.cpp:536
(gdb) p mDecoder
Cannot access memory at address 0x440e0508
** Root Cause and Solution **
RtspMediaResource (inherited BaseMediaResource) holds a raw pointer to mDecoder. This segmentation fault happens when mDecoder is already shutdown and destroyed by media element but RtspMediaResource still tries to use it.
We should set mDecoder as null pointer in RtspMediaResource::Close() and add checks for mDecoder wherever needed.
Assignee | ||
Comment 8•11 years ago
|
||
GDB log when the crash happened.
Assignee | ||
Comment 9•11 years ago
|
||
This patch fixes the problem that RTSP video app might crash after playing for a period of time and closing the app.
Attachment #8351144 -
Flags: review?(sworkman)
Assignee | ||
Comment 10•11 years ago
|
||
Hi Steve, this is another small fix for rtsp crash. Could you help to review it?
Updated•11 years ago
|
Attachment #8351144 -
Flags: review?(sworkman) → review+
Assignee | ||
Comment 11•11 years ago
|
||
Update the comment.
Attachment #8351144 -
Attachment is obsolete: true
Attachment #8355886 -
Flags: review+
Assignee | ||
Comment 12•11 years ago
|
||
The result of try server:
https://tbpl.mozilla.org/?tree=Try&rev=fb0455e3b222
Assignee | ||
Comment 13•11 years ago
|
||
Hi Steve, thanks for your review.
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 14•11 years ago
|
||
Keywords: checkin-needed
Comment 15•11 years ago
|
||
RTSP video is not a blocker for 1.3
blocking-b2g: 1.3? → -
Flags: needinfo?(skasetti)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 17•11 years ago
|
||
Nominated to 1.3+ for this is important bug fixing.
Hi Sri,
Some RTSP functions were already landed in 1.3 per BD purpose.
Is there any concern to uplift related bug fixing?
blocking-b2g: - → 1.3+
Flags: needinfo?(skasetti)
Comment 18•11 years ago
|
||
status-b2g-v1.3:
--- → fixed
status-firefox27:
--- → wontfix
status-firefox28:
--- → fixed
status-firefox29:
--- → fixed
Flags: needinfo?(skasetti)
Updated•11 years ago
|
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
Reporter | ||
Comment 19•11 years ago
|
||
Thanks for your help!
Verified it. I cannot reproduce this bug.
* Verificetion build:
- Gaia 1f029aa4c924d6c59fd7e8c6d9786a7370755d49
- Gecko http://hg.mozilla.org/releases/mozilla-aurora/rev/c42fd6b6d1fc
- BuildID 20140112004002
- Version 28.0a2
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 20•11 years ago
|
||
(In reply to William Hsu [:whsu] from comment #19)
> Verified it. I cannot reproduce this bug.
Great to hear that!
William, thanks for your work. We need your help to make RTSP feature more and more robust. :)
Reporter | ||
Comment 21•11 years ago
|
||
You're welcome!
Due to your hard working, the FxOS RTSP feature became more robust than before!
Thanks!
You need to log in
before you can comment on or make changes to this bug.
Description
•