cerc-io/solidity
16
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #6544 from ethereum/smt_warn_external

Browse Source 
[SMTChecker] Add note about not inlining external function calls
This commit is contained in:
chriseth 2019-04-18 12:14:53 +02:00 committed by GitHub
commit 3ba811f1ad
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libsolidity/formal/SMTChecker.cpp
View File

@ -121,6 +121,7 @@ bool SMTChecker::visit(FunctionDefinition const& _function)
initializeLocalVariables(_function);
m_loopExecutionHappened = false;
m_arrayAssignmentHappened = false;
m_externalFunctionCallHappened = false;
}
_function.parameterList().accept(*this);
if (_function.returnParameterList())
@ -607,6 +608,7 @@ void SMTChecker::endVisit(FunctionCall const& _funCall)
popCallStack();
break;
case FunctionType::Kind::External:
m_externalFunctionCallHappened = true;
resetStateVariables();
resetStorageReferences();
break;
@ -1295,6 +1297,12 @@ void SMTChecker::checkCondition(
" therefore all mapping information is erased after"
" a mapping local variable/parameter is assigned.\n"
"You can re-introduce information using require().";
if (m_externalFunctionCallHappened)
extraComment +=
"\nNote that external function calls are not inlined,"
" even if the source code of the function is available."
" This is due to the possibility that the actual called contract"
" has the same ABI but implements the function differently.";
SecondarySourceLocation secondaryLocation{};
secondaryLocation.append(extraComment, SourceLocation{});

1
libsolidity/formal/SMTChecker.h
View File

@ -277,6 +277,7 @@ private:
VariableUsage m_variableUsage;
bool m_loopExecutionHappened = false;
bool m_arrayAssignmentHappened = false;
bool m_externalFunctionCallHappened = false;
// True if the "No SMT solver available" warning was already created.
bool m_noSolverWarning = false;
/// An Expression may have multiple smt::Expression due to