2009-01-31 20:29:27 +01:00
|
|
|
/*
|
|
|
|
* Cppcheck - A tool for static C/C++ code analysis
|
2011-01-09 20:33:36 +01:00
|
|
|
* Copyright (C) 2007-2011 Daniel Marjamäki and Cppcheck team.
|
2009-01-31 20:29:27 +01:00
|
|
|
*
|
|
|
|
* 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
|
2009-09-27 17:08:31 +02:00
|
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
2009-01-31 20:29:27 +01:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
|
|
//---------------------------------------------------------------------------
|
|
|
|
#ifndef CheckBufferOverrunH
|
|
|
|
#define CheckBufferOverrunH
|
|
|
|
//---------------------------------------------------------------------------
|
|
|
|
|
2009-03-20 17:35:53 +01:00
|
|
|
#include "check.h"
|
2009-02-11 06:16:10 +01:00
|
|
|
#include "settings.h"
|
2010-11-21 11:48:27 +01:00
|
|
|
#include "mathlib.h"
|
2009-02-11 06:16:10 +01:00
|
|
|
#include <list>
|
2010-02-14 19:58:17 +01:00
|
|
|
#include <vector>
|
|
|
|
#include <string>
|
2009-02-11 06:16:10 +01:00
|
|
|
|
|
|
|
class ErrorLogger;
|
|
|
|
class Token;
|
|
|
|
class Tokenizer;
|
2011-06-23 04:44:11 +02:00
|
|
|
class Variable;
|
2009-01-31 20:29:27 +01:00
|
|
|
|
2009-07-17 10:49:01 +02:00
|
|
|
/// @addtogroup Checks
|
|
|
|
/// @{
|
|
|
|
|
2010-03-13 21:42:59 +01:00
|
|
|
/**
|
|
|
|
* @brief buffer overruns and array index out of bounds
|
|
|
|
*
|
|
|
|
* Buffer overrun and array index out of bounds are pretty much the same.
|
|
|
|
* But I generally use 'array index' if the code contains []. And the given
|
|
|
|
* index is out of bounds.
|
|
|
|
* I generally use 'buffer overrun' if you for example call a strcpy or
|
|
|
|
* other function and pass a buffer and reads or writes too much data.
|
|
|
|
*/
|
2011-10-13 20:53:06 +02:00
|
|
|
class CheckBufferOverrun : public Check {
|
2009-01-31 20:29:27 +01:00
|
|
|
public:
|
2009-03-20 17:35:53 +01:00
|
|
|
|
2009-03-21 07:53:23 +01:00
|
|
|
/** This constructor is used when registering the CheckClass */
|
2011-02-02 10:29:10 +01:00
|
|
|
CheckBufferOverrun() : Check(myName())
|
2009-03-20 17:35:53 +01:00
|
|
|
{ }
|
|
|
|
|
2010-03-17 22:16:18 +01:00
|
|
|
/** This constructor is used when running checks. */
|
2009-07-13 16:00:15 +02:00
|
|
|
CheckBufferOverrun(const Tokenizer *tokenizer, const Settings *settings, ErrorLogger *errorLogger)
|
2011-02-02 10:29:10 +01:00
|
|
|
: Check(myName(), tokenizer, settings, errorLogger)
|
2009-03-20 17:35:53 +01:00
|
|
|
{ }
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void runSimplifiedChecks(const Tokenizer *tokenizer, const Settings *settings, ErrorLogger *errorLogger) {
|
2009-07-13 16:00:15 +02:00
|
|
|
CheckBufferOverrun checkBufferOverrun(tokenizer, settings, errorLogger);
|
2009-10-13 21:39:51 +02:00
|
|
|
checkBufferOverrun.bufferOverrun();
|
2010-04-18 20:51:39 +02:00
|
|
|
checkBufferOverrun.negativeIndex();
|
2011-08-04 11:15:14 +02:00
|
|
|
checkBufferOverrun.arrayIndexThenCheck();
|
2010-04-10 21:12:00 +02:00
|
|
|
|
|
|
|
/** ExecutionPath checking.. */
|
|
|
|
checkBufferOverrun.executionPaths();
|
2009-03-20 17:35:53 +01:00
|
|
|
}
|
2009-01-31 20:29:27 +01:00
|
|
|
|
2010-03-13 21:42:59 +01:00
|
|
|
/** @brief %Check for buffer overruns */
|
2009-01-31 20:29:27 +01:00
|
|
|
void bufferOverrun();
|
2010-03-13 21:42:59 +01:00
|
|
|
|
2011-08-04 11:15:14 +02:00
|
|
|
/** @brief Using array index before bounds check */
|
|
|
|
void arrayIndexThenCheck();
|
|
|
|
|
2010-04-10 21:12:00 +02:00
|
|
|
/** @brief %Check for buffer overruns by inspecting execution paths */
|
|
|
|
void executionPaths();
|
|
|
|
|
2010-03-13 21:42:59 +01:00
|
|
|
/**
|
|
|
|
* @brief Get minimum length of format string result
|
|
|
|
* @param input_string format string
|
|
|
|
* @param parameters given parameters to sprintf
|
|
|
|
* @return minimum length of resulting string
|
|
|
|
*/
|
2010-12-31 09:30:56 +01:00
|
|
|
static MathLib::bigint countSprintfLength(const std::string &input_string, const std::list<const Token*> ¶meters);
|
2009-09-25 18:23:44 +02:00
|
|
|
|
2009-10-07 14:37:20 +02:00
|
|
|
/**
|
2010-03-13 21:42:59 +01:00
|
|
|
* @brief %Check code that matches: "sprintf ( %varid% , %str% [,)]" when varid is not 0,
|
2009-10-07 14:37:20 +02:00
|
|
|
* and report found errors.
|
|
|
|
* @param tok The "sprintf" token.
|
|
|
|
* @param size The size of the buffer where sprintf is writing.
|
|
|
|
*/
|
2010-12-31 09:30:56 +01:00
|
|
|
void checkSprintfCall(const Token *tok, const MathLib::bigint size);
|
2009-10-07 14:37:20 +02:00
|
|
|
|
2009-01-31 20:29:27 +01:00
|
|
|
/** Check for buffer overruns - locate struct variables and check them with the .._CheckScope function */
|
2009-07-05 22:16:43 +02:00
|
|
|
void checkStructVariable();
|
2009-01-31 20:29:27 +01:00
|
|
|
|
2009-07-05 22:16:43 +02:00
|
|
|
/** Check for buffer overruns - locate global variables and local function variables and check them with the checkScope function */
|
|
|
|
void checkGlobalAndLocalVariable();
|
2009-01-31 20:29:27 +01:00
|
|
|
|
2010-05-26 10:56:34 +02:00
|
|
|
/** Check for buffer overruns due to allocating strlen(src) bytes instead of (strlen(src)+1) bytes before copying a string */
|
|
|
|
void checkBufferAllocatedWithStrlen();
|
2010-06-02 18:09:25 +02:00
|
|
|
|
2010-06-02 07:41:07 +02:00
|
|
|
/** Check for buffer overruns due to copying command-line args to fixed-sized buffers without bounds checking */
|
|
|
|
void checkInsecureCmdLineArgs();
|
2010-05-26 10:56:34 +02:00
|
|
|
|
2010-04-18 20:51:39 +02:00
|
|
|
/** Check for negative index */
|
|
|
|
void negativeIndex();
|
|
|
|
|
2010-04-18 11:08:29 +02:00
|
|
|
/** Information about N-dimensional array */
|
2011-10-13 20:53:06 +02:00
|
|
|
class ArrayInfo {
|
2010-04-18 11:08:29 +02:00
|
|
|
private:
|
|
|
|
/** number of elements of array */
|
2010-12-31 09:30:56 +01:00
|
|
|
std::vector<MathLib::bigint> _num;
|
2010-04-18 11:08:29 +02:00
|
|
|
|
|
|
|
/** size of each element in array */
|
2010-12-31 09:30:56 +01:00
|
|
|
MathLib::bigint _element_size;
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2010-04-18 19:46:45 +02:00
|
|
|
/** variable id */
|
|
|
|
unsigned int _varid;
|
|
|
|
|
2010-04-18 11:08:29 +02:00
|
|
|
/** full name of variable as pattern */
|
|
|
|
std::string _varname;
|
|
|
|
|
|
|
|
public:
|
|
|
|
ArrayInfo();
|
|
|
|
ArrayInfo(const ArrayInfo &);
|
2011-06-23 04:44:11 +02:00
|
|
|
ArrayInfo(const Variable *var, const Tokenizer *tokenizer);
|
2010-09-15 22:25:12 +02:00
|
|
|
ArrayInfo & operator=(const ArrayInfo &ai);
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2010-04-21 18:33:21 +02:00
|
|
|
/**
|
|
|
|
* Create array info with specified data
|
|
|
|
* The intention is that this is only a temporary solution.. all
|
|
|
|
* checking should be based on ArrayInfo from the start and then
|
|
|
|
* this will not be needed as the declare can be used instead.
|
|
|
|
*/
|
2010-12-31 09:30:56 +01:00
|
|
|
ArrayInfo(unsigned int id, const std::string &name, MathLib::bigint size1, MathLib::bigint n);
|
2010-04-21 18:33:21 +02:00
|
|
|
|
2010-04-24 21:48:58 +02:00
|
|
|
/** Create a copy ArrayInfo where the number of elements have been limited by a value */
|
2010-11-21 11:48:27 +01:00
|
|
|
ArrayInfo limit(MathLib::bigint value) const;
|
2010-04-24 21:48:58 +02:00
|
|
|
|
2010-04-18 11:08:29 +02:00
|
|
|
/**
|
|
|
|
* Declare array - set info
|
2010-04-18 19:46:45 +02:00
|
|
|
* \param tok first token in array declaration
|
|
|
|
* \param tokenizer The tokenizer (for type size)
|
2010-04-18 11:08:29 +02:00
|
|
|
* \return success => true
|
|
|
|
*/
|
2010-04-18 19:46:45 +02:00
|
|
|
bool declare(const Token *tok, const Tokenizer &tokenizer);
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2011-06-23 02:35:58 +02:00
|
|
|
/** array sizes */
|
2011-10-13 20:53:06 +02:00
|
|
|
const std::vector<MathLib::bigint> &num() const {
|
2011-06-23 02:35:58 +02:00
|
|
|
return _num;
|
|
|
|
}
|
|
|
|
|
2010-04-18 11:08:29 +02:00
|
|
|
/** array size */
|
2011-11-22 21:14:14 +01:00
|
|
|
MathLib::bigint num(std::size_t index) const {
|
2011-06-23 02:35:58 +02:00
|
|
|
return _num[index];
|
|
|
|
}
|
2011-11-22 21:14:14 +01:00
|
|
|
void num(std::size_t index, MathLib::bigint number) {
|
2011-09-10 16:14:32 +02:00
|
|
|
_num[index] = number;
|
|
|
|
}
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2010-04-18 20:18:25 +02:00
|
|
|
/** size of each element */
|
2011-10-13 20:53:06 +02:00
|
|
|
MathLib::bigint element_size() const {
|
2011-06-23 02:35:58 +02:00
|
|
|
return _element_size;
|
|
|
|
}
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2010-04-18 19:46:45 +02:00
|
|
|
/** Variable name */
|
2011-10-13 20:53:06 +02:00
|
|
|
unsigned int varid() const {
|
2011-06-23 02:35:58 +02:00
|
|
|
return _varid;
|
|
|
|
}
|
2011-10-13 20:53:06 +02:00
|
|
|
void varid(unsigned int id) {
|
2011-09-10 17:21:52 +02:00
|
|
|
_varid = id;
|
|
|
|
}
|
2010-04-18 19:46:45 +02:00
|
|
|
|
2010-04-18 11:08:29 +02:00
|
|
|
/** Variable name */
|
2011-10-13 20:53:06 +02:00
|
|
|
const std::string &varname() const {
|
2011-06-23 02:35:58 +02:00
|
|
|
return _varname;
|
|
|
|
}
|
2011-10-13 20:53:06 +02:00
|
|
|
void varname(const std::string &name) {
|
2011-09-12 01:21:13 +02:00
|
|
|
_varname = name;
|
|
|
|
}
|
2010-04-18 11:08:29 +02:00
|
|
|
};
|
|
|
|
|
2010-04-21 18:33:21 +02:00
|
|
|
/** Check for buffer overruns (based on ArrayInfo) */
|
|
|
|
void checkScope(const Token *tok, const ArrayInfo &arrayInfo);
|
|
|
|
|
2011-09-10 17:21:52 +02:00
|
|
|
/** Check for buffer overruns */
|
|
|
|
void checkScope(const Token *tok, const std::vector<std::string> &varname, const ArrayInfo &arrayInfo);
|
|
|
|
|
2011-02-10 21:56:06 +01:00
|
|
|
/** Check scope helper function - parse for body */
|
|
|
|
void checkScopeForBody(const Token *tok, const ArrayInfo &arrayInfo, bool &bailout);
|
2010-04-21 18:33:21 +02:00
|
|
|
|
|
|
|
/** Helper function used when parsing for-loops */
|
|
|
|
void parse_for_body(const Token *tok2, const ArrayInfo &arrayInfo, const std::string &strindex, bool condition_out_of_bounds, unsigned int counter_varid, const std::string &min_counter_value, const std::string &max_counter_value);
|
|
|
|
|
2011-10-24 21:22:04 +02:00
|
|
|
/** Check readlink or readlinkat() buffer usage */
|
|
|
|
void checkReadlinkBufferUsage(const Token *tok, const Token *scope_begin, const MathLib::bigint total_size, const bool is_readlinkat);
|
|
|
|
|
2010-04-21 20:02:58 +02:00
|
|
|
/**
|
2011-01-22 21:31:26 +01:00
|
|
|
* Helper function for checkFunctionCall - check a function parameter
|
2010-04-21 20:02:58 +02:00
|
|
|
* \param tok token for the function name
|
|
|
|
* \param par on what parameter is the array used
|
|
|
|
* \param arrayInfo the array information
|
2011-12-11 08:16:58 +01:00
|
|
|
* \param callstack call stack. This is used to prevent recursion and to provide better error messages. Pass a empty list from checkScope etc.
|
2010-04-21 20:02:58 +02:00
|
|
|
*/
|
2011-12-11 08:16:58 +01:00
|
|
|
void checkFunctionParameter(const Token &tok, const unsigned int par, const ArrayInfo &arrayInfo, std::list<const Token *> callstack);
|
2011-01-22 21:31:26 +01:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Helper function that checks if the array is used and if so calls the checkFunctionCall
|
|
|
|
* @param tok token that matches "%var% ("
|
|
|
|
* @param arrayInfo the array information
|
2011-12-11 08:16:58 +01:00
|
|
|
* \param callstack call stack. This is used to prevent recursion and to provide better error messages. Pass a empty list from checkScope etc.
|
2011-01-22 21:31:26 +01:00
|
|
|
*/
|
2011-12-11 08:16:58 +01:00
|
|
|
void checkFunctionCall(const Token *tok, const ArrayInfo &arrayInfo, std::list<const Token *> callstack);
|
2010-04-18 11:08:29 +02:00
|
|
|
|
2011-08-24 13:11:39 +02:00
|
|
|
void arrayIndexOutOfBoundsError(const Token *tok, const ArrayInfo &arrayInfo, const std::vector<MathLib::bigint> &index);
|
|
|
|
void arrayIndexOutOfBoundsError(const std::list<const Token *> &callstack, const ArrayInfo &arrayInfo, const std::vector<MathLib::bigint> &index);
|
|
|
|
void bufferOverrunError(const Token *tok, const std::string &varnames = "");
|
2011-12-11 08:16:58 +01:00
|
|
|
void bufferOverrunError(const std::list<const Token *> &callstack, const std::string &varnames = "");
|
2011-08-24 13:11:39 +02:00
|
|
|
void strncatUsageError(const Token *tok);
|
2011-10-05 20:17:57 +02:00
|
|
|
void outOfBoundsError(const Token *tok, const std::string &what, const bool show_size_info, const MathLib::bigint &supplied_size, const MathLib::bigint &actual_size);
|
2011-08-24 13:11:39 +02:00
|
|
|
void sizeArgumentAsCharError(const Token *tok);
|
2011-09-05 21:59:41 +02:00
|
|
|
void terminateStrncpyError(const Token *tok, const std::string &varname);
|
|
|
|
void bufferNotZeroTerminatedError(const Token *tok, const std::string &varname, const std::string &function);
|
2010-11-21 11:48:27 +01:00
|
|
|
void negativeIndexError(const Token *tok, MathLib::bigint index);
|
2010-06-02 07:41:07 +02:00
|
|
|
void cmdLineArgsError(const Token *tok);
|
2011-08-24 13:11:39 +02:00
|
|
|
void pointerOutOfBoundsError(const Token *tok, const std::string &object); // UB when result of calculation is out of bounds
|
2011-08-04 11:15:14 +02:00
|
|
|
void arrayIndexThenCheckError(const Token *tok, const std::string &indexName);
|
2011-08-21 21:18:41 +02:00
|
|
|
void possibleBufferOverrunError(const Token *tok, const std::string &src, const std::string &dst, bool cat);
|
2011-10-24 21:22:04 +02:00
|
|
|
void possibleReadlinkBufferOverrunError(const Token *tok, const std::string &funcname, const std::string &varname);
|
2009-03-22 08:20:15 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void getErrorMessages(ErrorLogger *errorLogger, const Settings *settings) {
|
2010-12-29 12:43:29 +01:00
|
|
|
CheckBufferOverrun c(0, settings, errorLogger);
|
2011-09-11 15:54:26 +02:00
|
|
|
std::vector<MathLib::bigint> indexes;
|
|
|
|
indexes.push_back(2);
|
|
|
|
c.arrayIndexOutOfBoundsError(0, ArrayInfo(0, "array", 1, 2), indexes);
|
2011-08-24 13:11:39 +02:00
|
|
|
c.bufferOverrunError(0, std::string("buffer"));
|
|
|
|
c.strncatUsageError(0);
|
2011-10-05 20:17:57 +02:00
|
|
|
c.outOfBoundsError(0, "index", true, 2, 1);
|
2011-08-24 13:11:39 +02:00
|
|
|
c.sizeArgumentAsCharError(0);
|
2011-09-05 21:59:41 +02:00
|
|
|
c.terminateStrncpyError(0, "buffer");
|
|
|
|
c.bufferNotZeroTerminatedError(0, "buffer", "strncpy");
|
2010-12-29 12:43:29 +01:00
|
|
|
c.negativeIndexError(0, -1);
|
|
|
|
c.cmdLineArgsError(0);
|
2011-08-24 13:11:39 +02:00
|
|
|
c.pointerOutOfBoundsError(0, "array");
|
2011-08-04 11:15:14 +02:00
|
|
|
c.arrayIndexThenCheckError(0, "index");
|
2011-08-21 21:18:41 +02:00
|
|
|
c.possibleBufferOverrunError(0, "source", "destination", false);
|
2011-10-24 21:22:04 +02:00
|
|
|
c.possibleReadlinkBufferOverrunError(0, "readlink", "buffer");
|
2009-03-22 08:20:15 +01:00
|
|
|
}
|
2009-06-12 12:19:37 +02:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
std::string myName() const {
|
2009-06-12 15:20:08 +02:00
|
|
|
return "Bounds checking";
|
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
std::string classInfo() const {
|
2009-06-12 12:19:37 +02:00
|
|
|
return "out of bounds checking";
|
|
|
|
}
|
2009-01-31 20:29:27 +01:00
|
|
|
};
|
2009-07-17 10:49:01 +02:00
|
|
|
/// @}
|
2009-01-31 20:29:27 +01:00
|
|
|
//---------------------------------------------------------------------------
|
|
|
|
#endif
|
|
|
|
|
2009-09-25 18:23:44 +02:00
|
|
|
|
|
|
|
|