361 lines
11 KiB
C++
361 lines
11 KiB
C++
/*
|
|
* c++check - c/c++ syntax checking
|
|
* Copyright (C) 2007-2008 Daniel Marjamäki and Reijo Tomperi
|
|
*
|
|
* This program is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/
|
|
*/
|
|
#include "cppcheck.h"
|
|
|
|
#include "preprocessor.h" // preprocessor.
|
|
#include "tokenize.h" // <- Tokenizer
|
|
|
|
#include "CheckMemoryLeak.h"
|
|
#include "CheckBufferOverrun.h"
|
|
#include "CheckClass.h"
|
|
#include "CheckHeaders.h"
|
|
#include "CheckOther.h"
|
|
#include "CheckFunctionUsage.h"
|
|
#include "FileLister.h"
|
|
|
|
#include <algorithm>
|
|
#include <sstream>
|
|
#include <cstring>
|
|
#include <fstream>
|
|
#include <map>
|
|
|
|
//---------------------------------------------------------------------------
|
|
|
|
CppCheck::CppCheck( ErrorLogger &errorLogger ) : _checkFunctionUsage( this )
|
|
{
|
|
_errorLogger = &errorLogger;
|
|
}
|
|
|
|
CppCheck::~CppCheck()
|
|
{
|
|
|
|
}
|
|
|
|
void CppCheck::settings( const Settings &settings )
|
|
{
|
|
_settings = settings;
|
|
}
|
|
|
|
void CppCheck::addFile( const std::string &path )
|
|
{
|
|
_filenames.push_back( path );
|
|
}
|
|
|
|
void CppCheck::addFile( const std::string &path, const std::string &content )
|
|
{
|
|
_filenames.push_back( path );
|
|
_fileContents[ path ] = content;
|
|
}
|
|
|
|
std::string CppCheck::parseFromArgs( int argc, const char* const argv[] )
|
|
{
|
|
std::vector<std::string> pathnames;
|
|
bool Recursive = false;
|
|
|
|
for (int i = 1; i < argc; i++)
|
|
{
|
|
// Flag used for various purposes during debugging
|
|
if (strcmp(argv[i],"--debug") == 0)
|
|
_settings._debug = true;
|
|
|
|
// Show all messages
|
|
else if (strcmp(argv[i],"--all") == 0)
|
|
_settings._showAll = true;
|
|
|
|
// Only print something when there are errors
|
|
else if (strcmp(argv[i],"--errorsonly")==0)
|
|
_settings._errorsOnly = true;
|
|
|
|
// Checking coding style
|
|
else if (strcmp(argv[i],"--style")==0)
|
|
_settings._checkCodingStyle = true;
|
|
|
|
// Recursively check source files
|
|
else if (strcmp(argv[i],"--recursive")==0)
|
|
Recursive = true;
|
|
|
|
// Verbose error messages (configuration info)
|
|
else if (strcmp(argv[i],"--verbose")==0)
|
|
_settings._verbose = true;
|
|
|
|
else
|
|
pathnames.push_back( argv[i] );
|
|
}
|
|
|
|
// --recursive was used
|
|
if ( Recursive )
|
|
{
|
|
if( pathnames.size() == 0 )
|
|
{
|
|
// Handle situation: cppcheck --recursive
|
|
FileLister::RecursiveAddFiles( _filenames, "", true );
|
|
}
|
|
else
|
|
{
|
|
// Handle situation: cppcheck --recursive path1 path2
|
|
|
|
// Execute RecursiveAddFiles() to each given file parameter
|
|
std::vector<std::string>::const_iterator iter;
|
|
for(iter=pathnames.begin(); iter!=pathnames.end(); iter++)
|
|
FileLister::RecursiveAddFiles( _filenames, iter->c_str(), true );
|
|
}
|
|
}
|
|
else
|
|
{
|
|
std::vector<std::string>::const_iterator iter;
|
|
for(iter=pathnames.begin(); iter!=pathnames.end(); iter++)
|
|
FileLister::RecursiveAddFiles( _filenames, iter->c_str(), false );
|
|
}
|
|
|
|
if (_filenames.empty())
|
|
{
|
|
std::ostringstream oss;
|
|
oss << "c++check 1.26\n"
|
|
"\n"
|
|
"C/C++ code checking\n"
|
|
"\n"
|
|
"Syntax:\n"
|
|
" cppcheck [--all] [--errorsonly] [--recursive] [--style] [--verbose] [filename1] [filename2]\n"
|
|
"\n"
|
|
"Options:\n"
|
|
" --all Make the checking more sensitive. More bugs are detected,\n"
|
|
" but there are also more false positives\n"
|
|
" --errorsonly Only print error messages\n"
|
|
" --recursive Recursively check all *.cpp, *.cxx, *.cc and *.c files\n"
|
|
" --style Check coding style\n"
|
|
" --verbose More detailed error reports\n";
|
|
return oss.str();
|
|
}
|
|
|
|
// Check function usage if "--style" and "--all" was given.
|
|
// There will be false positives for exported library functions
|
|
if ( _settings._showAll && _settings._checkCodingStyle )
|
|
_settings._checkFunctionUsage = true;
|
|
|
|
return "";
|
|
}
|
|
|
|
void CppCheck::check()
|
|
{
|
|
std::sort( _filenames.begin(), _filenames.end() );
|
|
for (unsigned int c = 0; c < _filenames.size(); c++)
|
|
{
|
|
_errout.str("");
|
|
std::string fname = _filenames[c];
|
|
|
|
// If only errors are printed, print filename after the check
|
|
if ( _settings._errorsOnly == false )
|
|
_errorLogger->reportOut( std::string( "Checking " ) + fname + std::string( "..." ) );
|
|
|
|
Preprocessor preprocessor( this );
|
|
std::map<std::string, std::string> code;
|
|
if( _fileContents.size() > 0 && _fileContents.find( _filenames[c] ) != _fileContents.end() )
|
|
{
|
|
// File content was given as a string
|
|
std::istringstream iss( _fileContents[ _filenames[c] ] );
|
|
preprocessor.preprocess(iss, code, fname);
|
|
}
|
|
else
|
|
{
|
|
// Only file name was given, read the content from file
|
|
std::ifstream fin( fname.c_str() );
|
|
preprocessor.preprocess(fin, code, fname);
|
|
}
|
|
|
|
for ( std::map<std::string,std::string>::const_iterator it = code.begin(); it != code.end(); ++it )
|
|
{
|
|
cfg = it->first;
|
|
checkFile(it->second, _filenames[c].c_str());
|
|
}
|
|
|
|
if ( _settings._errorsOnly == false && _errout.str().empty() )
|
|
_errorLogger->reportOut( "No errors found" );
|
|
}
|
|
|
|
// This generates false positives - especially for libraries
|
|
_settings._verbose = false;
|
|
if ( _settings._checkFunctionUsage )
|
|
{
|
|
_errout.str("");
|
|
if( _settings._errorsOnly == false )
|
|
_errorLogger->reportOut( "Checking usage of global functions (this may take several minutes).." );
|
|
|
|
_checkFunctionUsage.check();
|
|
}
|
|
|
|
}
|
|
|
|
|
|
//---------------------------------------------------------------------------
|
|
// CppCheck - A function that checks a specified file
|
|
//---------------------------------------------------------------------------
|
|
|
|
void CppCheck::checkFile(const std::string &code, const char FileName[])
|
|
{
|
|
Tokenizer _tokenizer;
|
|
|
|
// Tokenize the file
|
|
{
|
|
std::istringstream istr(code);
|
|
_tokenizer.tokenize(istr, FileName);
|
|
}
|
|
|
|
_tokenizer.fillFunctionList();
|
|
|
|
// Check that the memsets are valid.
|
|
// The 'memset' function can do dangerous things if used wrong.
|
|
// Important: The checking doesn't work on simplified tokens list.
|
|
CheckClass checkClass( &_tokenizer, _settings, this );
|
|
checkClass.CheckMemset();
|
|
|
|
|
|
// Check for unsigned divisions where one operand is signed
|
|
// Very important to run it before 'SimplifyTokenList'
|
|
CheckOther checkOther( &_tokenizer, this );
|
|
if ( _settings._checkCodingStyle )
|
|
checkOther.CheckUnsignedDivision();
|
|
|
|
// Give warning when using char variable as array index
|
|
// Doesn't work on simplified token list ('unsigned')
|
|
if ( _settings._checkCodingStyle )
|
|
checkOther.CheckCharVariable();
|
|
|
|
|
|
// Including header which is not needed (too many false positives)
|
|
// if ( _settings._checkCodingStyle )
|
|
// {
|
|
// CheckHeaders checkHeaders( &tokenizer );
|
|
// checkHeaders.WarningIncludeHeader();
|
|
// }
|
|
|
|
|
|
_tokenizer.simplifyTokenList();
|
|
|
|
|
|
if ( _settings._checkFunctionUsage )
|
|
_checkFunctionUsage.parseTokens(_tokenizer);
|
|
|
|
// Memory leak
|
|
CheckMemoryLeakClass checkMemoryLeak( &_tokenizer, _settings, this );
|
|
checkMemoryLeak.CheckMemoryLeak();
|
|
|
|
// Buffer overruns..
|
|
CheckBufferOverrunClass checkBufferOverrun( &_tokenizer, _settings, this );
|
|
checkBufferOverrun.CheckBufferOverrun();
|
|
|
|
// Check that all class constructors are ok.
|
|
checkClass.CheckConstructors();
|
|
|
|
// Check that all base classes have virtual destructors
|
|
checkClass.virtualDestructor();
|
|
|
|
if (_settings._showAll)
|
|
{
|
|
// Check for "if (a=b)"
|
|
checkOther.CheckIfAssignment();
|
|
|
|
// Check for case without break
|
|
// Disabled because it generates many false positives
|
|
// CheckCaseWithoutBreak();
|
|
|
|
// Dangerous usage of strtok
|
|
// Disabled because it generates false positives
|
|
//WarningStrTok();
|
|
}
|
|
|
|
|
|
|
|
// Dangerous functions, such as 'gets' and 'scanf'
|
|
checkBufferOverrun.WarningDangerousFunctions();
|
|
|
|
|
|
// Invalid function usage..
|
|
checkOther.InvalidFunctionUsage();
|
|
|
|
|
|
if (_settings._checkCodingStyle)
|
|
{
|
|
// Check that all private functions are called.
|
|
checkClass.CheckUnusedPrivateFunctions();
|
|
|
|
// Warning upon c-style pointer casts
|
|
const char *ext = strrchr(FileName, '.');
|
|
if (ext && strcmp(ext,".cpp")==0)
|
|
checkOther.WarningOldStylePointerCast();
|
|
|
|
// Use standard functions instead
|
|
checkOther.WarningIsDigit();
|
|
checkOther.WarningIsAlpha();
|
|
|
|
checkClass.CheckOperatorEq1();
|
|
|
|
// if (a) delete a;
|
|
checkOther.WarningRedundantCode();
|
|
|
|
// if (condition);
|
|
checkOther.WarningIf();
|
|
|
|
// Variable scope (check if the scope could be limited)
|
|
//CheckVariableScope();
|
|
|
|
// Check if a constant function parameter is passed by value
|
|
checkOther.CheckConstantFunctionParameter();
|
|
|
|
// Unused struct members..
|
|
checkOther.CheckStructMemberUsage();
|
|
|
|
// Check for various types of incomplete statements that could for example
|
|
// mean that an ';' has been added by accident
|
|
checkOther.CheckIncompleteStatement();
|
|
|
|
// Unreachable code below a 'return' statement
|
|
checkOther.unreachableCode();
|
|
|
|
// Usage of local functions
|
|
checkOther.functionVariableUsage();
|
|
}
|
|
}
|
|
//---------------------------------------------------------------------------
|
|
|
|
void CppCheck::reportErr( const std::string &errmsg)
|
|
{
|
|
if ( /*OnlyReportUniqueErrors*/ true )
|
|
{
|
|
if ( std::find( _errorList.begin(), _errorList.end(), errmsg ) != _errorList.end() )
|
|
return;
|
|
_errorList.push_back( errmsg );
|
|
}
|
|
|
|
std::string errmsg2( errmsg );
|
|
if ( _settings._verbose )
|
|
{
|
|
errmsg2 += "\n Defines=\'" + cfg + "\'\n";
|
|
}
|
|
|
|
|
|
_errorLogger->reportErr( errmsg2 );
|
|
|
|
_errout << errmsg2 << std::endl;
|
|
}
|
|
|
|
void CppCheck::reportOut( const std::string &outmsg)
|
|
{
|
|
// This is currently never called. It is here just to comply with
|
|
// the interface.
|
|
}
|