cppcheck/src/checkautovariables.cpp

226 lines
6.9 KiB
C++
Raw Normal View History

/*
* Cppcheck - A tool for static C/C++ code analysis
* Copyright (C) 2007-2009 Daniel Marjamäki, Reijo Tomperi, Nicolas Le Cam,
* Leandro Penz, Kimmo Varis, Vesa Pikki, Gianluca Scacco
*
* 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/
*/
//---------------------------------------------------------------------------
// Buffer overrun..
//---------------------------------------------------------------------------
#include "checkautovariables.h"
#include "tokenize.h"
#include <algorithm>
#include <sstream>
#include <list>
#include <cstring>
#include <iostream>
#include <string>
#include <cstdlib> // <- strtoul
//---------------------------------------------------------------------------
// Register this check class into cppcheck by creating a static instance of it..
namespace
{
static CheckAutoVariables instance;
}
// _callStack used when parsing into subfunctions.
2009-03-22 08:01:48 +01:00
bool CheckAutoVariables::errorAv(const Token* left, const Token* right)
{
std::string left_var = left->str();
std::string right_var = right->str();
std::list<std::string>::iterator it_fp;
2009-03-22 08:01:48 +01:00
for (it_fp = fp_list.begin();it_fp != fp_list.end();++it_fp)
{
std::string vname = (*it_fp);
2009-03-22 08:01:48 +01:00
2009-03-22 08:23:20 +01:00
//The left argument is a formal parameter
if (vname == left_var)
{
//cout << "Beccato" << endl;
2009-03-22 08:01:48 +01:00
break;
}
}
2009-03-22 08:01:48 +01:00
//The left argument is NOT a formal parameter
if (it_fp == fp_list.end())
2009-03-22 08:01:48 +01:00
return false;
std::list<std::string>::iterator id_vd;
2009-03-22 08:01:48 +01:00
for (id_vd = vd_list.begin();id_vd != vd_list.end();++id_vd)
{
std::string vname = (*id_vd);
2009-03-22 08:23:20 +01:00
//The left argument is a variable declaration
if (vname == right_var)
2009-03-22 08:01:48 +01:00
break;
}
2009-03-22 08:01:48 +01:00
//The left argument is NOT a variable declaration
if (id_vd == vd_list.end())
2009-03-22 08:01:48 +01:00
return false;
//If I reach this point there is a wrong assignement of an auto-variable to an effective parameter of a function
return true;
}
2009-03-22 08:01:48 +01:00
bool CheckAutoVariables::isAutoVar(const Token* t)
{
std::list<std::string>::iterator id_vd;
std::string v = t->str();
2009-03-22 08:01:48 +01:00
for (id_vd = vd_list.begin();id_vd != vd_list.end();++id_vd)
{
std::string vname = (*id_vd);
if (vname == v)
return true;
}
return false;
}
void print(const Token *tok, int num)
{
const Token *t = tok;
std::cout << tok->linenr() << " PRINT ";
for (int i = 0;i < num;i++)
{
std::cout << " [" << t->str() << "] ";
t = t->next();
}
std::cout << std::endl;
}
bool isTypeName(const Token *tok)
{
bool ret = false;
std::string _str = tok->str();
const char *type[] = {"case", "return", "delete", 0};
for (int i = 0; type[i]; i++)
ret |= (_str == type[i]);
return !ret;
}
bool isStatic(const Token *tok)
{
bool res = false;
if (Token::Match(tok->tokAt(-1), "static "))
res = true;
else if (Token::Match(tok->tokAt(-2), "static "))
res = true;
else if (Token::Match(tok->tokAt(-3), "static "))
res = true;
//std::cout << __PRETTY_FUNCTION__ << " " << tok->str() << " " << res << std::endl;
return res;
}
void CheckAutoVariables::addVD(const Token* tok)
{
std::string var_name;
var_name = tok->str();
//std::cout << "VD " << tok->linenr() << " " << var_name << std::endl;
vd_list.push_back(var_name);
}
void CheckAutoVariables::autoVariables()
{
bool begin_function = false;
bool begin_function_decl = false;
int bindent = 0;
for (const Token *tok = _tokenizer->tokens(); tok; tok = tok->next())
{
if (Token::Match(tok, "%type% %var% (") ||
Token::Match(tok, "%type% * %var% (") ||
Token::Match(tok, "%type% :: %var% ("))
{
begin_function = true;
fp_list.clear();
vd_list.clear();
}
else if (begin_function && begin_function_decl && Token::Match(tok, "%type% * %var%"))
{
std::string var_name;
var_name = tok->tokAt(2)->str();
//cout << "FP " << tok->linenr() << " " << var_name << endl;
fp_list.push_back(var_name);
}
else if (begin_function && Token::Match(tok, "("))
begin_function_decl = true;
else if (begin_function && Token::Match(tok, ")"))
{
begin_function_decl = false;
}
else if (begin_function && Token::Match(tok, "{"))
bindent++;
else if (begin_function && Token::Match(tok, "}"))
{
bindent--;
}
else if (bindent > 0 && Token::Match(tok, "%type% :: %any%") && !isStatic(tok)) //Inside a function
{
addVD(tok->tokAt(2));
}
else if (bindent > 0 && Token::Match(tok, "%var% %var% ;") && !isStatic(tok)) //Inside a function
{
if (!isTypeName(tok))
continue;
addVD(tok->tokAt(1));
}
else if (bindent > 0 && Token::Match(tok, "const %var% %var% ;") && !isStatic(tok)) //Inside a function
{
if (!isTypeName(tok->tokAt(1)))
continue;
addVD(tok->tokAt(2));
}
else if (bindent > 0 && Token::Match(tok, "%var% = & %var%")) //Critical assignement
{
2009-03-22 08:01:48 +01:00
if (errorAv(tok->tokAt(0), tok->tokAt(3)))
2009-03-21 21:33:27 +01:00
reportError(tok,
"error",
"autoVariables",
"Wrong assignement of an auto-variable to an effective parameter of a function");
}
else if (bindent > 0 && Token::Match(tok, "%var% [ %any% ] = & %var%")) //Critical assignement
{
2009-03-22 08:01:48 +01:00
if (errorAv(tok->tokAt(0), tok->tokAt(6)))
2009-03-21 21:33:27 +01:00
reportError(tok,
"error",
"autoVariables",
"Wrong assignement of an auto-variable to an effective parameter of a function");
}
else if (bindent > 0 && Token::Match(tok, "return & %var%")) //Critical return
{
2009-03-22 08:01:48 +01:00
if (isAutoVar(tok->tokAt(2)))
2009-03-21 21:33:27 +01:00
reportError(tok,
"error",
"autoVariables",
"Return of the address of an auto-variable");
}
}
vd_list.clear();
fp_list.clear();
}
//---------------------------------------------------------------------------