[FFmpeg-devel] Coverity

Stefan Gehrer stefan.gehrer
Thu May 29 08:30:15 CEST 2008


I noticed and thanks for fixing the bug in my code (cavsdec.c).
I am usually skeptical about static code analysis, but that was
a serious bug the tool found.


Michael Niedermayer wrote:
> Hi
> As everyone should have already noticed (from svn log) we are currently
> trying to fix all the coverity warnings (also called "defects").
> 24 are left, are the maintainers of the respective files going to deal with
> them or should i try?
> Whats left:
> USE_AFTER_FREE 	matroska_parse_info  		libavformat/matroskadec.c
> USE_AFTER_FREE 	matroska_add_stream 		libavformat/matroskadec.c
> USE_AFTER_FREE 	ebml_read_header 		libavformat/matroskadec.c
> UNINIT 		rtp_new_av_stream 		ffserver.c
> RESOURCE_LEAK 	parse_ffconfig 			ffserver.c
> RESOURCE_LEAK 	sdt_cb 				libavformat/mpegts.c
> RESOURCE_LEAK 	sdt_cb 				libavformat/mpegts.c
> RESOURCE_LEAK 	pat_cb 				libavformat/mpegts.c
> RESOURCE_LEAK 	matroska_parse_block 		libavformat/matroskadec.c
> RESOURCE_LEAK 	matroska_add_stream 		libavformat/matroskadec.c
> RESOURCE_LEAK 	mpegts_write_header 		libavformat/mpegtsenc.c
> NULL_RETURNS 	nuv_header 			libavformat/nuv.c
> NULL_RETURNS 	nuv_header 			libavformat/nuv.c
> NULL_RETURNS 	get_sindex 			libavformat/gxf.c
> NULL_RETURNS 	daud_header 			libavformat/daud.c
> NEGATIVE_RETURNS sdp_parse_fmtp_config_h264 	libavformat/rtp_h264.c
> NEGATIVE_RETURNS matroska_read_header 		libavformat/matroskadec.c
> FORWARD_NULL 	rtsp_read_packet 		libavformat/rtsp.c
> FORWARD_NULL 	sdp_read_header 		libavformat/rtsp.c
> FORWARD_NULL 	qdm2_decode_fft_packets 	libavcodec/qdm2.c
> FORWARD_NULL 	parse_ffconfig 			ffserver.c
> FORWARD_NULL 	parse_ffconfig 			ffserver.c
> FORWARD_NULL 	parse_ffconfig 			ffserver.c
> FORWARD_NULL 	vorbis_parse_setup_hdr_codebooks libavcodec/vorbis_dec.c

More information about the ffmpeg-devel mailing list