Only S_OK hresult from GetModuleMetaData() guarantees return of valid pointer

diff --git a/src/info/classinfo.cpp b/src/info/classinfo.cpp
index 1a40b23..bd603cc 100644 (file)
--- a/src/info/classinfo.cpp
+++ b/src/info/classinfo.cpp
@@ -367,7 +367,7 @@ __forceinline HRESULT ClassInfo::InitializeRegularClassName(
         hr = info.v1()->GetModuleMetaData(
             this->moduleId, ofRead, IID_IMetaDataImport,
             reinterpret_cast<IUnknown**>(&pMDImport));
-        if (FAILED(hr))
+        if (hr != S_OK)
         {
             throw HresultException(
                 "ClassInfo::InitializeRegularClassName(): "

diff --git a/src/info/functioninfo.cpp b/src/info/functioninfo.cpp
index 1cf211c..65e370b 100644 (file)
--- a/src/info/functioninfo.cpp
+++ b/src/info/functioninfo.cpp
@@ -793,7 +793,7 @@ HRESULT FunctionInfo::Initialize(
         hr = info.v1()->GetModuleMetaData(
             this->moduleId, ofRead, IID_IMetaDataImport,
             reinterpret_cast<IUnknown**>(&pMDImport));
-        if (FAILED(hr))
+        if (hr != S_OK)
         {
             throw HresultException(
                 "FunctionInfo::Initialize(): GetModuleMetaData()", hr);