Closed Bug 1731583 Opened 3 years ago Closed 2 years ago

Crash in [@ mozalloc_abort | abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu]

Categories

(Core :: WebRTC: Audio/Video, defect, P3)

Unspecified
macOS
defect

Tracking

()

RESOLVED DUPLICATE of bug 1654112
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- unaffected
firefox92 --- unaffected
firefox93 --- unaffected
firefox94 --- wontfix
firefox95 --- wontfix

People

(Reporter: calixte, Assigned: jhlin)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: crash, regression)

Crash Data

Maybe Fission related. (DOMFissionEnabled=1)

Crash report: https://crash-stats.mozilla.org/report/index/b5affbcb-5dd5-4bf4-a783-716510210920

MOZ_CRASH Reason: MOZ_CRASH()

Top 10 frames of crashing thread:

0 libmozglue.dylib mozalloc_abort memory/mozalloc/mozalloc_abort.cpp:33
1 libmozglue.dylib abort memory/mozalloc/mozalloc_abort.cpp:86
2 XUL rtc::FatalMessage::~FatalMessage third_party/libwebrtc/webrtc/rtc_base/checks.cc:69
3 XUL rtc::FatalMessage::~FatalMessage third_party/libwebrtc/webrtc/rtc_base/checks.cc:63
4 XUL webrtc::RtpPacketizerH264::PacketizeSingleNalu third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:296
5 XUL webrtc::RtpPacketizerH264::GeneratePackets third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:178
6 XUL webrtc::RtpPacketizerH264::SetPayloadData third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:170
7 XUL webrtc::RTPSenderVideo::SendVideo third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc:387
8 XUL webrtc::RTPSender::SendOutgoingData third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_sender.cc:454
9 XUL webrtc::ModuleRtpRtcpImpl::SendOutgoingData third_party/libwebrtc/webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc:462

There are 2 crashes (from 1 installation) in nightly 94 with buildid 20210919212908. In analyzing the backtrace, the regression may have been introduced by patch [1] to fix bug 1665776.

[1] https://hg.mozilla.org/mozilla-central/rev?node=3e842ac77d06

Flags: needinfo?(jolin)

(In reply to Calixte Denizet (:calixte) from comment #0)

There are 2 crashes (from 1 installation) in nightly 94 with buildid 20210919212908. In analyzing the backtrace, the regression may have been introduced by patch [1] to fix bug 1665776.

[1] https://hg.mozilla.org/mozilla-central/rev?node=3e842ac77d06

Yes, it is caused by bug 1665776. Some H.264 platform encoder returns a frame larger than the max payload size and triggers the check. A fix is under development.

Assignee: nobody → jolin
Flags: needinfo?(jolin)
Priority: -- → P1

Set release status flags based on info from the regressing bug 1665776

John, did you have a chance to look into this more?

Flags: needinfo?(jolin)
Crash Signature: [@ mozalloc_abort | abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu] → [@ mozalloc_abort | abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu] [@ abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu]
Has Regression Range: --- → yes

John, that looks fixed, is that right ?

Crash Signature: [@ mozalloc_abort | abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu] [@ abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu] → [@ mozalloc_abort | abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu] [@ abort | webrtc::RtpPacketizerH264::PacketizeSingleNalu]
Flags: needinfo?(jolin)
Flags: needinfo?(jolin)
Severity: S2 → S4
Priority: P1 → P3

:jib, could you answer the questions for John?

Flags: needinfo?(jib)

Bug 1654112 removed the offending RTC_CHECK_GE(payload_size_left and no crash reports >95 so this appears fixed.

Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(jib)
Resolution: --- → DUPLICATE
Flags: needinfo?(jolin)
You need to log in before you can comment on or make changes to this bug.