Anonymous | Login | Signup for a new account | 2021-01-22 09:08 UTC | ![]() |
Main | My View | View Issues | Roadmap | Repositories |
View Issue Details [ Jump to Notes ] | [ Print ] | |||||||
ID | Project | Category | View Status | Date Submitted | Last Update | |||
0000563 | Tunix | General Features | public | 2012-04-05 17:48 | 2012-04-05 22:24 | |||
Reporter | Fred | |||||||
Assigned To | dandruczyk | |||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | |||
Status | closed | Resolution | fixed | |||||
Platform | All | OS | All | OS Version | All | |||
Product Version | 0.9.24-SNAPSHOT | |||||||
Target Version | 0.9.24 | Fixed in Version | 0.9.24 | |||||
Summary | 0000563: Consider Checking 64 bit happiness using this | |||||||
Description | http://sourceforge.net/apps/mediawiki/cppcheck/index.php?title=Main_Page#64-bit_portability [^] Came across it while looking for something for the firmware and thought maybe it would help with your 64 efforts. | |||||||
Tags | No tags attached. | |||||||
FirmwareVersion | nananana | |||||||
Attached Files | ||||||||
![]() |
|
(0001392) dandruczyk (viewer) 2012-04-05 21:38 |
DL'd it, ran it, fixed reported issues. No portability issues found, only a few minor memory leaks, all rsolve as of commit d8e1328e7f7fb8a0af6ee92abe773dc907506872 Checking 3d_vetable.c... [3d_vetable.c:2428]: (error) Memory leak: table_list 10/117 files checked 8% done Checking dataio.c... [dataio.c:416]: (error) Possible null pointer dereference: error 14/117 files checked 11% done Checking gui_handlers.c... [gui_handlers.c:1895]: (error) Memory leak: prefix 83/117 files checked 70% done Checking plugins/mscommon/vex_support.c... [plugins/mscommon/vex_support.c:920]: (error) Memory leak: result [plugins/mscommon/vex_support.c:1041]: (error) Memory leak: result |
![]() Fred (administrator) 2012-04-05 22:08 |
Sweet! A useful tool for the belt, then :-) If you run it over the old code before 64 fixes will it find anything? If you have time and are also curious, I'd be interested in the results of that experiment :-) |
(0001394) dandruczyk (viewer) 2012-04-05 22:19 |
I didn't DL it, i used the one available via apt-get (mine is probably a little dated). I suggest for your curiosity that you write a script to pull each hash, and run cppcheck across it with whatever options and then graph the resulting error counts per file. I don't have that kind of free time, or that curiosity about the past. |
![]() Fred (administrator) 2012-04-05 22:24 |
"i used the one available via apt-get" - me too, just now, found nothing in the firmware, but that doesn't mean much. Closing, recommend you run it periodically. And any other similar stuff you can find that works and is even better. |
Copyright © 2000 - 2011 MantisBT Group |