diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2017-03-18 11:12:35 +0100 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2017-03-18 11:07:43 +0000 |
| commit | 468d2964e9c30df4cdbfcab000b1f513021a22d3 (patch) | |
| tree | 07c454005c4053181011659c336cd33df1fa9cf1 | |
| parent | c154f9a0827dcc2fc69870198d368fa8ebb4cde2 (diff) | |
cppcheck: nullPointer
Avoid to forget E_POINTER return value
Change-Id: I1887d20c7d73dcd3c6ddc1287b6c1ad6c13dd69f
Reviewed-on: https://gerrit.libreoffice.org/35394
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
| -rw-r--r-- | extensions/source/ole/unoobjw.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extensions/source/ole/unoobjw.cxx b/extensions/source/ole/unoobjw.cxx index 0262723d5eeb..63c7cf30272d 100644 --- a/extensions/source/ole/unoobjw.cxx +++ b/extensions/source/ole/unoobjw.cxx @@ -1060,7 +1060,7 @@ HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned s { bHandled= true; if( !pvarResult) - ret= E_POINTER; + return E_POINTER; CComObject< JScriptValue>* pValue; if( SUCCEEDED( CComObject<JScriptValue>::CreateInstance( &pValue))) { @@ -1104,7 +1104,7 @@ HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned s { bHandled= true; if( !pvarResult) - ret= E_POINTER; + return E_POINTER; // the first parameter is in DISPPARAMS rgvargs contains the name of the struct. CComVariant arg; if( pdispparams->cArgs != 1) |