@@ -238,22 +238,22 @@ void CheckNullPointer::parseFunctionCall(const Token &tok, std::list<const Token
238238 var.push_back (firstParam);
239239 else if (value == 0 && Token::Match (&tok, " snprintf|vsnprintf|fnprintf|vfnprintf" ) && secondParam && secondParam->str () != " 0" ) // Only if length (second parameter) is not zero
240240 var.push_back (firstParam);
241- else if (value == 0 && library != NULL && library->isnullargbad (tok.str (),1 ))
242- var.push_back (firstParam);
243- else if (value == 1 && library != NULL && library->isuninitargbad (tok.str (),1 ))
244- var.push_back (firstParam);
245- }
241+ else if (value == 0 && library != NULL && library->isnullargbad (tok.str (),1 ))
242+ var.push_back (firstParam);
243+ else if (value == 1 && library != NULL && library->isuninitargbad (tok.str (),1 ))
244+ var.push_back (firstParam);
245+ }
246246
247247 // 2nd parameter..
248248 if ((value == 0 && Token::Match (secondParam, " 0|NULL ,|)" )) || (secondParam && secondParam->varId () > 0 )) {
249249 if (functionNames2_all.find (tok.str ()) != functionNames2_all.end ())
250250 var.push_back (secondParam);
251251 else if (value == 0 && functionNames2_nullptr.find (tok.str ()) != functionNames2_nullptr.end ())
252252 var.push_back (secondParam);
253- else if (value == 0 && library != NULL && library->isnullargbad (tok.str (),2 ))
254- var.push_back (secondParam);
255- else if (value == 1 && library != NULL && library->isuninitargbad (tok.str (),2 ))
256- var.push_back (secondParam);
253+ else if (value == 0 && library != NULL && library->isnullargbad (tok.str (),2 ))
254+ var.push_back (secondParam);
255+ else if (value == 1 && library != NULL && library->isuninitargbad (tok.str (),2 ))
256+ var.push_back (secondParam);
257257 }
258258
259259 if (Token::Match (&tok, " printf|sprintf|snprintf|fprintf|fnprintf|scanf|sscanf|fscanf|wprintf|swprintf|fwprintf|wscanf|swscanf|fwscanf" )) {
@@ -817,7 +817,7 @@ void CheckNullPointer::nullPointerByDeRefAndChec()
817817
818818 if (Token::Match (tok2->next (), " %var% ( %varid% ," , varid)) {
819819 std::list<const Token *> varlist;
820- parseFunctionCall (*(tok2->next ()), varlist, &_settings->library , 0 );
820+ parseFunctionCall (*(tok2->next ()), varlist, &_settings->library , 0 );
821821 if (!varlist.empty () && varlist.front () == tok2->tokAt (3 )) {
822822 nullPointerError (tok2->tokAt (3 ), varname, tok, inconclusive);
823823 break ;
@@ -1365,13 +1365,13 @@ class Nullpointer : public ExecutionPath {
13651365
13661366private:
13671367 const SymbolDatabase* symbolDatabase;
1368- const Library *library;
1368+ const Library *library;
13691369
13701370 /* * Create checking of specific variable: */
13711371 Nullpointer (Check *c, const unsigned int id, const std::string &name, const SymbolDatabase* symbolDatabase_, const Library *lib)
13721372 : ExecutionPath(c, id),
13731373 symbolDatabase (symbolDatabase_),
1374- library(lib),
1374+ library(lib),
13751375 varname(name),
13761376 null(false ) {
13771377 }
0 commit comments