From 468d2964e9c30df4cdbfcab000b1f513021a22d3 Mon Sep 17 00:00:00 2001 From: Julien Nabet Date: Sat, 18 Mar 2017 11:12:35 +0100 Subject: cppcheck: nullPointer Avoid to forget E_POINTER return value Change-Id: I1887d20c7d73dcd3c6ddc1287b6c1ad6c13dd69f Reviewed-on: https://gerrit.libreoffice.org/35394 Tested-by: Jenkins Reviewed-by: Julien Nabet --- extensions/source/ole/unoobjw.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'extensions') 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::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) -- cgit