Merge pull request #118 from Microsoft/develop

Create release
microsoft · Oct 31, 2017 · f1a9c6c · f1a9c6c
2 parents 44e1483 + fc96978
commit f1a9c6c
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 13 deletions.
diff --git a/appveyor.yml b/appveyor.yml
@@ -35,7 +35,21 @@ test_script:
   - tools\test.cmd -v
 
 artifacts:
+  - path: bin\$(configuration)\vswhere.exe
+    name: vswhere
+
   - path: bin\$(configuration)
     type: zip
 
+deploy:
+  - provider: GitHub
+    auth_token:
+      secure: 5K94QY8dHKlbjvlBpQaXwAkmDf+LQE3g5WlU9snnNMjDjf0UT4xDoOBCLWcmwZOm
+    artifact: vswhere
+    prerelease: true
+    on:
+      # Signed releases will come from VSTS on master
+      branch: develop
+      configuration: Release
+
 # vim: set ai et st=2 sts=2 sw=2:
diff --git a/src/vswhere.lib/Formatter.cpp b/src/vswhere.lib/Formatter.cpp
@@ -411,7 +411,7 @@ HRESULT Formatter::GetProductId(_In_ ISetupInstance* pInstance, _Out_ VARIANT* p
         ISetupPackageReferencePtr reference;
 
         hr = instance->GetProduct(&reference);
-        if (SUCCEEDED(hr))
+        if (SUCCEEDED(hr) && !!reference)
         {
             variant_t vt;
 
@@ -437,7 +437,7 @@ HRESULT Formatter::GetProductPath(_In_ ISetupInstance* pInstance, _Out_ VARIANT*
         bstr_t bstrProductPath;
 
         hr = instance->GetProductPath(bstrProductPath.GetAddress());
-        if (SUCCEEDED(hr))
+        if (SUCCEEDED(hr) && !!bstrProductPath)
         {
             variant_t vt;
 

diff --git a/src/vswhere.lib/InstanceSelector.cpp b/src/vswhere.lib/InstanceSelector.cpp
@@ -210,7 +210,7 @@ bool InstanceSelector::IsProductMatch(_In_ ISetupInstance2* pInstance) const
     ISetupPackageReferencePtr product;
 
     auto hr = pInstance->GetProduct(&product);
-    if (FAILED(hr))
+    if (FAILED(hr) || !product)
     {
         // Should always have a product so no match.
         return false;

diff --git a/src/vswhere.lib/vswhere.lib.rc b/src/vswhere.lib/vswhere.lib.rc
diff --git a/test/vswhere.test/TestInstance.h b/test/vswhere.test/TestInstance.h
@@ -232,20 +232,15 @@ class TestInstance :
         _Outptr_result_maybenull_ ISetupPackageReference** ppPackage
     )
     {
-        if (m_product)
-        {
-            *ppPackage = m_product;
-            return S_OK;
-        }
-
-        return E_NOTFOUND;
+        *ppPackage = m_product;
+        return S_OK;
     }
 
     STDMETHODIMP GetProductPath(
         _Outptr_result_maybenull_ BSTR* pbstrProductPath
     )
     {
-        return TryGetBSTR(L"ProductPath", pbstrProductPath);
+        return TryGetBSTR(L"ProductPath", pbstrProductPath, TRUE);
     }
 
     STDMETHODIMP GetErrors(
@@ -297,7 +292,7 @@ class TestInstance :
         _Outptr_result_maybenull_ BSTR* pbstrEnginePath
     )
     {
-        return TryGetBSTR(L"EnginePath", pbstrEnginePath);
+        return TryGetBSTR(L"EnginePath", pbstrEnginePath, TRUE);
     }
 
     // ISetupInstanceCatalog
@@ -372,7 +367,7 @@ class TestInstance :
         return E_NOTFOUND;
     }
 
-    STDMETHODIMP TryGetBSTR(_In_ LPCWSTR wszName, _Out_ BSTR* pbstrValue)
+    STDMETHODIMP TryGetBSTR(_In_ LPCWSTR wszName, _Out_ BSTR* pbstrValue, _In_opt_ BOOL fAllowNull = FALSE)
     {
         if (!pbstrValue)
         {
@@ -390,6 +385,11 @@ class TestInstance :
                 return E_OUTOFMEMORY;
             }
         }
+        else if (fAllowNull && E_NOTFOUND == hr)
+        {
+            *pbstrValue = NULL;
+            return S_OK;
+        }
 
         return hr;
     }

diff --git a/tools/Publish-Release.ps1 b/tools/Publish-Release.ps1
@@ -65,6 +65,8 @@ if ($Commit) {
 
 if ($Name) {
     $values.name = $Name
+} else {
+    $values.name = $Tag
 }
 
 if ($Description) {