Closed
Bug 1651286
Opened 4 years ago
Closed 4 years ago
PeerConnectionImpl.cpp: PeerConnectionImpl::CreatePeerConnection() appears to dereference NULL
Categories
(Core :: WebRTC, defect, P2)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla80
Tracking | Status | |
---|---|---|
firefox80 | --- | fixed |
People
(Reporter: izbyshev, Assigned: mjf)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Unless I'm mistaken, the call at https://searchfox.org/mozilla-central/rev/7ec7ee4a/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp#238 always crashes because of the following:
- PeerConnectionImpl ctor has
aGlobal
parameter which isnullptr
by default - The definition of that ctor passes
aGlobal
toRTCStatsTimestampMaker
ctor which dereferences it.
Reporter | ||
Updated•4 years ago
|
Blocks: svace-analysis
Updated•4 years ago
|
Component: Source Code Analysis → WebRTC
Product: Firefox Build System → Core
Assignee | ||
Comment 1•4 years ago
|
||
Based on what I see in searchfox[1], that static method isn't used anywhere. I'll remove it.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=777c78bcfafcb90995f07473067e53e5ad68887f
Assignee: nobody → mfroman
Severity: -- → S3
Priority: -- → P2
Assignee | ||
Comment 2•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Pushed by mfroman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1f31aaa73c38 remove unused static method PeerConnectionImpl::CreatePeerConnection. r=bwc
Comment 4•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in
before you can comment on or make changes to this bug.
Description
•