diff options
| author | Rusty Wagner <rusty@vector35.com> | 2018-03-16 20:07:48 -0400 |
|---|---|---|
| committer | Peter LaFosse <peter@vector35.com> | 2018-03-23 17:10:06 -0400 |
| commit | 5ed96234690937b8487a79d11fc30e7bb39b48ec (patch) | |
| tree | 57d162d1d532cc87535b9d5eccb3014a636beb0b | |
| parent | 062d1b508982f310b97efc34ddc59ddb04c3f869 (diff) | |
Fix reference leak in calling conventions
| -rw-r--r-- | callingconvention.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/callingconvention.cpp b/callingconvention.cpp index 2eb57567..a56b8f11 100644 --- a/callingconvention.cpp +++ b/callingconvention.cpp @@ -195,8 +195,10 @@ void CallingConvention::GetIncomingVariableForParameterVariableCallback(void* ct BNFunction* func, BNVariable* result) { CallingConvention* cc = (CallingConvention*)ctxt; - *result = cc->GetIncomingVariableForParameterVariable(*var, - func ? new Function(BNNewFunctionReference(func)) : nullptr); + Ref<Function> funcObj; + if (func) + funcObj = new Function(BNNewFunctionReference(func)); + *result = cc->GetIncomingVariableForParameterVariable(*var, funcObj); } @@ -204,8 +206,10 @@ void CallingConvention::GetParameterVariableForIncomingVariableCallback(void* ct BNFunction* func, BNVariable* result) { CallingConvention* cc = (CallingConvention*)ctxt; - *result = cc->GetParameterVariableForIncomingVariable(*var, - func ? new Function(BNNewFunctionReference(func)) : nullptr); + Ref<Function> funcObj; + if (func) + funcObj = new Function(BNNewFunctionReference(func)); + *result = cc->GetParameterVariableForIncomingVariable(*var, funcObj); } |
