Automated Code Analysis

cppcheck analysis of mplayer_1.0~rc3++final.dfsg1-1.dsc

• ./TOOLS/asfinfo.c:271 [error] - Resource leak: f
• ./TOOLS/avisubdump.c:181 [error] - Resource leak: f
• ./TOOLS/bmovl-test.c:119 [error] - Resource leak: fifo
• ./TOOLS/movinfo.c:340 [error] - Resource leak: f
• stream/stream_netstream.h:97 [error] - Common realloc mistake: "pack" nulled but not freed upon failure
• stream/stream_netstream.h:103 [error] - Memory leak: pack
• ./TOOLS/vivodump.c:187 [error] - Resource leak: f
• ./drivers/tdfx_vid_test.c:58 [error] - Resource leak: fd
• ./get_path.c:111 [error] - Common realloc mistake: "res_url_path" nulled but not freed upon failure
• ./get_path.c:117 [error] - Common realloc mistake: "bdl_url_path" nulled but not freed upon failure
• ./gui/bitmap.c:47 [error] - Resource leak: fp
• ./gui/interface.c:498 [error] - Common realloc mistake: "vf_settings" nulled but not freed upon failure
• ./gui/mplayer/gtk/pl.c:290 [error] - Common realloc mistake: "CLListSelected" nulled but not freed upon failure
• ./gui/skin/font.c:122 [error] - Resource leak: f
• ./gui/win32/gui.c:1548 [error] - Memory leak: skinName
• ./gui/win32/skinload.c:548 [error] - Memory leak: filename
• ./gui/win32/skinload.c:548 [error] - Memory leak: tmp
• ./gui/win32/skinload.c:548 [error] - Memory leak: desc
• ./gui/win32/skinload.c:640 [error] - Memory leak: filename
• ./gui/win32/skinload.c:640 [error] - Memory leak: tmp
• ./gui/win32/skinload.c:640 [error] - Memory leak: desc
• ./input/lirc.c:98 [error] - Common realloc mistake: "cmd_buf" nulled but not freed upon failure
• ./libaf/af_ladspa.c:541 [error] - Memory leak: buf
• ./libass/ass_render.c:2533 [error] - Uninitialized variable: next2
• ./libavcodec/ffv1.c:781 [error] - Array index -1 is out of bounds
• ./libavcodec/ffv1.c:813 [error] - Array index -1 is out of bounds
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: ''.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ALIGNED_BITSTREAM_WRITER;ALT_BITSTREAM_WRITER'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ALIGNED_BITSTREAM_WRITER;ALT_BITSTREAM_WRITER;ARCH_X86'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ALT_BITSTREAM_READER_LE'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ALT_BITSTREAM_WRITER'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ALT_BITSTREAM_WRITER;ARCH_X86'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_ARM'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_ARM;HAVE_ARMV6'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_ARM;HAVE_NEON'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_BFIN'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_PPC'.
• ./libavcodec/g729dec.c:125 [error] - Invalid number of character ({) when these macros are defined: 'ARCH_SH4'.
• ./libavcodec/indeo3.c:91 [error] - Array 'corrector_type_2[8]' index -248 out of bounds
• ./libavcodec/mdct.c:51 [error] - Array 'ff_sine_128[128]' index 128 out of bounds
• ./libavcodec/mdct.c:52 [error] - Array 'ff_sine_256[256]' index 256 out of bounds
• ./libavcodec/mdct.c:53 [error] - Array 'ff_sine_512[512]' index 512 out of bounds
• ./libavcodec/mdct.c:54 [error] - Array 'ff_sine_1024[1024]' index 1024 out of bounds
• ./libavcodec/mdct.c:55 [error] - Array 'ff_sine_2048[2048]' index 2048 out of bounds
• ./libavcodec/mdct.c:56 [error] - Array 'ff_sine_4096[4096]' index 4096 out of bounds
• ./libavcodec/x86/dsputil_mmx.c:47 [error] - Array 'ff_pdw_80000000[2]' index 2 out of bounds
• ./libavcodec/x86/dsputil_mmx.c:74 [error] - Array 'ff_pd_1[2]' index 2 out of bounds
• ./libavcodec/x86/dsputil_mmx.c:75 [error] - Array 'ff_pd_2[2]' index 2 out of bounds
• ./libavformat/asfenc.c:339 [error] - Possible null pointer dereference: title - otherwise it is redundant to check if title is null at line 344
• ./libavformat/iss.c:76 [error] - Uninitialized variable: token
• ./libavformat/iss.c:77 [error] - Uninitialized variable: token
• ./libavformat/mpegts.c:1318 [error] - Uninitialized variable: packet
• ./libdvdnav/vm/vm.c:213 [error] - Deallocating a deallocated pointer: fd
• ./libfaad2/sbr_noise.h:32 [error] - Invalid number of character ({) when these macros are defined: 'DRM_PS;__cplusplus'.
• ./libfaad2/sbr_qmf_c.h:32 [error] - Invalid number of character ({) when these macros are defined: 'DRM_PS;__cplusplus'.
• ./libmenu/menu.c:200 [error] - Common realloc mistake: "menu_list" nulled but not freed upon failure
• ./libmenu/menu.c:255 [error] - Common realloc mistake: "buffer" nulled but not freed upon failure
• ./libmpcodecs/ad_libdca.c:252 [error] - Uninitialized variable: flags
• ./libmpcodecs/ad_realaud.c:233 [error] - Common realloc mistake: "path" nulled but not freed upon failure
• ./libmpcodecs/ad_speex.c:44 [error] - Memory leak: ctx
• ./libmpcodecs/mp_image.c:50 [error] - Memory leak: mpi
• ./libmpcodecs/vd.c:221 [error] - Possible null pointer dereference: vp - otherwise it is redundant to check if vp is null at line 223
• ./libmpcodecs/vf_remove_logo.c:544 [error] - Resource leak: input
• ./libmpcodecs/vf_remove_logo.c:541 [error] - Memory leak: new_pgm
• ./libmpdemux/demux_avs.c:225 [error] - Memory leak: dp
• ./libmpdemux/demux_film.c:116 [error] - Memory leak: dp
• ./libmpdemux/demux_gif.c:154 [error] - Memory leak: dp
• ./libmpdemux/demux_gif.c:246 [error] - Memory leak: priv
• ./libmpdemux/demux_mf.c:48 [error] - Memory leak: dp
• ./libmpdemux/demux_nut.c:232 [error] - Memory leak: dp
• ./libmpdemux/demux_rawdv.c:111 [error] - Memory leak: dp_video
• ./libmpdemux/demux_rawdv.c:150 [error] - Memory leak: frames
• ./libmpdemux/demux_rtp.cpp:605 [error] - Mismatching allocation and deallocation: ReadBufferQueue::fTag
• ./libmpdemux/demux_rtp.cpp:369 [error] - Possible null pointer dereference: env
• ./libmpdemux/demux_ts.c:2539 [error] - Memory leak: tss
• ./libmpdemux/demux_ty_osd.c:709 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:713 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:722 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:726 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:728 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:730 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:732 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:809 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:812 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:813 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:816 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:826 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/demux_ty_osd.c:827 [error] - Undefined behaviour: line is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./libmpdemux/muxer_mpeg.c:331 [error] - Memory leak: tmp
• ./libvo/gl_common.c:450 [error] - Memory leak: data
• ./libvo/vo_dxr2.c:920 [error] - Memory leak: uCode
• ./libvo/vo_dxr3.c:1135 [error] - Memory leak: lut
• ./libvo/vo_fbdev.c:616 [error] - Possible null pointer dereference: red - otherwise it is redundant to check if red is null at line 617
• ./libvo/vo_fbdev.c:624 [error] - Possible null pointer dereference: green - otherwise it is redundant to check if green is null at line 625
• ./libvo/vo_fbdev.c:633 [error] - Possible null pointer dereference: blue - otherwise it is redundant to check if blue is null at line 634
• ./libvo/vo_fbdev2.c:121 [error] - Possible null pointer dereference: red - otherwise it is redundant to check if red is null at line 122
• ./libvo/vo_fbdev2.c:129 [error] - Possible null pointer dereference: green - otherwise it is redundant to check if green is null at line 130
• ./libvo/vo_fbdev2.c:138 [error] - Possible null pointer dereference: blue - otherwise it is redundant to check if blue is null at line 139
• ./libvo/vo_vdpau.c:648 [error] - Common realloc mistake: "index_data" nulled but not freed upon failure
• ./libvo/vo_vdpau.c:763 [error] - Common realloc mistake: "eosd_surfaces" nulled but not freed upon failure
• ./libvo/vo_vdpau.c:764 [error] - Common realloc mistake: "eosd_targets" nulled but not freed upon failure
• ./loader/dmo/DMO_AudioDecoder.c:92 [error] - Memory leak: this.m_sVhdr
• ./loader/dmo/DMO_AudioDecoder.c:92 [error] - Memory leak: this.m_sVhdr2
• ./loader/dshow/DS_AudioDecoder.c:100 [error] - Memory leak: this.m_sVhdr
• ./loader/dshow/DS_AudioDecoder.c:100 [error] - Memory leak: this.m_sVhdr2
• ./m_option.c:69 [error] - Common realloc mistake: "p" nulled but not freed upon failure
• ./m_option.c:468 [error] - Common realloc mistake: "lst" nulled but not freed upon failure
• ./m_option.c:1659 [error] - Common realloc mistake: "obj_list" nulled but not freed upon failure
• ./m_property.c:169 [error] - Common realloc mistake: "ret" nulled but not freed upon failure
• ./mp3lib/test.c:71 [error] - Resource leak: f
• ./mp3lib/test2.c:69 [error] - Resource leak: audio_fd
• ./mplayer.c:493 [error] - Buffer access out-of-bounds
• ./mplayer.c:3064 [error] - Common realloc mistake: "psub" nulled but not freed upon failure
• ./osdep/macosx_finder_args.c:64 [error] - Common realloc mistake: "parm" nulled but not freed upon failure
• ./parser-mecmd.c:131 [error] - Common realloc mistake: "lst" nulled but not freed upon failure
• ./playtreeparser.c:240 [error] - Common realloc mistake: "e" nulled but not freed upon failure
• ./stream/cookies.c:112 [error] - Resource leak: fd
• ./stream/cookies.c:129 [error] - Memory leak: buffer
• ./stream/http.c:417 [error] - Common realloc mistake: "field" nulled but not freed upon failure
• ./stream/http.c:481 [error] - Memory leak: uri
• ./stream/stream_netstream.h:97 [error] - Common realloc mistake: "pack" nulled but not freed upon failure
• ./stream/stream_netstream.h:103 [error] - Memory leak: pack
• ./stream/stream_radio.c:1145 [error] - Memory leak: priv
• ./stream/tvi_dshow.c:476 [error] - Uninitialized variable: pBN
• ./stream/tvi_dshow.c:602 [error] - Uninitialized variable: buf
• ./stream/tvi_dshow.c:964 [error] - Uninitialized variable: pKSProp
• ./stream/tvi_dshow.c:1135 [error] - Uninitialized variable: pKSProp
• ./stream/tvi_dshow.c:1429 [error] - Uninitialized variable: pEnum
• ./stream/tvi_dshow.c:1432 [error] - Uninitialized variable: pVideoWindow
• ./stream/tvi_dshow.c:1881 [error] - Uninitialized variable: pPropBag
• ./stream/tvi_dshow.c:2732 [error] - Uninitialized variable: pTVAudio
• ./stream/tvi_dshow.c:2878 [error] - Uninitialized variable: pVideoWindow
• ./stream/tvi_dshow.c:2901 [error] - Uninitialized variable: pBF
• ./stream/tvi_dshow.c:3374 [error] - Uninitialized variable: pVD
• ./stream/tvi_dshow.c:3391 [error] - Uninitialized variable: pVD
• ./vidix/sh_veu_vid.c:155 [error] - Resource leak: fd

---