cppcheck: A tool for static C/C++ code analysis

everything about the current big update and the latest stable branch

Moderator: Moderator

lorenzodes
master
Posts: 772
Joined: Sun Mar 11, 2007 4:50 pm
Location: move.l 4.w,a6

cppcheck: A tool for static C/C++ code analysis

Postby lorenzodes » Sun Apr 05, 2009 3:30 pm

I have run cppcheck against the 1.1.1 branch of mms and it reported the following issues:

Code: Select all

[./common.cpp:450]: (error) Memory leak: np <------- false alarm
[./common.cpp:693]: (error) Memory leak: newbuf <------- false alarm
[./libs/ffmpeg/libswscale/swscale-example.c:229]: (error) Memory leak: rgb_data <-------  ignore, we don't maintain ffmpeg
[./libs/ffmpeg/libswscale/swscale-example.c:229]: (error) Memory leak: data <-------  ignore, we don't maintain ffmpeg
[./plugins/feature/audio/CD_Tag.cpp:349]: (error) Memory leak: np <------- false alarm
[./plugins/feature/audio/audio.hpp:45]: (error) Class Audio which is inherited by class AudioTemplate does not have a virtual destructor
[./plugins/feature/epg/epg.hpp:35]: (error) Class BaseChannel which is inherited by class Channel does not have a virtual destructor
[./plugins/feature/epg/epg.cpp:595]: (error) After push_back or push_front, the iterator 'it' may be invalid
[./plugins/feature/epg/epg.cpp:596]: (error) After push_back or push_front, the iterator 'it' may be invalid
[./plugins/feature/epg/epg.cpp:622]: (error) After push_back or push_front, the iterator 'it' may be invalid
[./plugins/feature/epg/epg.cpp:626]: (error) After push_back or push_front, the iterator 'it' may be invalid
[./plugins/feature/epg/epg.cpp:627]: (error) After push_back or push_front, the iterator 'it' may be invalid
[./plugins/feature/epg/epg.cpp:2202]: (error) After push_back or push_front, the iterator 'e' may be invalid
[./plugins/feature/epg/epg.cpp:2203]: (error) After push_back or push_front, the iterator 'e' may be invalid
[./plugins/feature/epg/epg.cpp:2205]: (error) After push_back or push_front, the iterator 'e' may be invalid
[./plugins/feature/epg/epg.cpp:2206]: (error) After push_back or push_front, the iterator 'e' may be invalid
[./plugins/feature/epg/epg.hpp:62]: (error) Class EpgEvent which is inherited by class SvdrpEvent does not have a virtual destructor
[./plugins/feature/epg/svdrp.cpp:902]: (error) After push_back or push_front, the iterator 'e' may be invalid
[./plugins/feature/movie/Movie_info.cpp:2405]: (error) Memory leak: np <------- false alarm
[./plugins/feature/movie/movie.hpp:208]: (error) Class MovieTemplate which is inherited by class MovieDB does not have a virtual destructor


Can everyone have a look at their code and check if those reported issues are real?
"I’m not frightened of dying, anytime will do, I don’t mind. Why should I be frightened of dying? There’s no reason for it, you gotta go sometime"

Return to “1.1.x”

Who is online

Users browsing this forum: No registered users and 2 guests