Bug 256432

Summary: Fix false-positive bad release after calling H264BufferToCMSampleBuffer/H265BufferToCMSampleBuffer
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: WebRTCAssignee: 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)
Reported 2023-05-07 12:16:51 PDT
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
Radar WebKit Bug Importer
Comment 1 2023-05-07 12:17:11 PDT
David Kilzer (:ddkilzer)
Comment 2 2023-05-07 12:26:36 PDT
David Kilzer (:ddkilzer)
Comment 3 2023-05-07 12:28:29 PDT
EWS
Comment 4 2023-05-08 06:48:32 PDT
Committed 263800@main (3163f4a6e89a): <https://commits.webkit.org/263800@main> Reviewed commits have been landed. Closing PR #13551 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.