Bug 256432
Summary: | Fix false-positive bad release after calling H264BufferToCMSampleBuffer/H265BufferToCMSampleBuffer | ||
---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> |
Component: | WebRTC | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | webkit-bug-importer, youennf |
Priority: | P2 | Keywords: | InRadar |
Version: | Other | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 246141 | ||
Bug Blocks: |
David Kilzer (:ddkilzer)
Fix false-positive bad release after calling H264BufferToCMSampleBuffer() and H265BufferToCMSampleBuffer().
Both of these methods return +1 retained CMSampleBufferRef objects, but are not instrumented with CF_RETURNS_RETAINED, so the clang static analyzer flags in the -decodeData:size:timeStamp: methods of RTCVideoDecoderH264 and RTCVideoDecoderH265.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/109014853>
David Kilzer (:ddkilzer)
Caused by:
Add support for AVC H264 WebCodecsVideoDecoder
<https://bugs.webkit.org/show_bug.cgi?id=246141>
<rdar://problem/100844377>
<https://commits.webkit.org/255422@main>
David Kilzer (:ddkilzer)
Pull request: https://github.com/WebKit/WebKit/pull/13551
EWS
Committed 263800@main (3163f4a6e89a): <https://commits.webkit.org/263800@main>
Reviewed commits have been landed. Closing PR #13551 and removing active labels.