CppCheck

Want to help the VideoLAN project? Designers, testers, translators, please come...
This is NOT about GETTING some help.
Forum rules
Please read the forum's rules carefully before posting. This forum should not be used to post VLC usage related questions.
nikola23
Cone that earned his stripes
Cone that earned his stripes
Posts: 115
Joined: 03 Feb 2009 07:25
VLC version: 1.2 pre3
Operating System: Win7\Ubuntu 10.10

CppCheck

Postby nikola23 » 02 Aug 2009 07:33

I was looking at the pidgin.im trac and someone found some memory leaks with cppcheck so i ran the VLC code against it and it found the following. Mind you I don't claim to know what the errors\warning mean, but I just hope to help.
[../modules/video_chroma/i420_yuy2.c:59]: (Error) Invalid number of character ({). Can't process file.
[../modules/video_filter/wall.c:425]: () Warning: Division with signed and unsigned operators
[../modules/video_filter/wall.c:426]: () Warning: Division with signed and unsigned operators
[../src/misc/threads.c:1323]: (Error) Memory leak: nfo
Also this is with the option to turn off possible false positives, which I'd estimate there were about 200.

ivoire
Cone that earned his stripes
Cone that earned his stripes
Posts: 413
Joined: 20 Aug 2008 11:29
VLC version: trunk
Operating System: linux (debian sid)
Contact:

Re: CppCheck

Postby ivoire » 04 Aug 2009 21:34

I can't reproduce it with the latest cppcheck. Moreover I'm wondering which version of vlc you are scanning as src/misc/threads.c doe not contain 1323 lines!

nikola23
Cone that earned his stripes
Cone that earned his stripes
Posts: 115
Joined: 03 Feb 2009 07:25
VLC version: 1.2 pre3
Operating System: Win7\Ubuntu 10.10

Re: CppCheck

Postby nikola23 » 05 Aug 2009 02:16

Sorry I stupidly ran it against the 1.0.1 code instead of a nightly copy.


Return to “Contribute and help the VideoLAN project”

Who is online

Users browsing this forum: No registered users and 29 guests