Huw Davies : winepulse: Return E_NOTIMPL when the property is not found.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b012bed6ff92a7716cd61647aea63772f5351fec
URL:    https://gitlab.winehq.org/wine/wine/-/commit/b012bed6ff92a7716cd61647aea6377...

Author: Huw Davies 
Date:   Mon Jun 12 10:23:53 2023 +0100

winepulse: Return E_NOTIMPL when the property is not found.

Fixes a regression caused by 13fa7a57df0d.

---

dlls/winepulse.drv/pulse.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/dlls/winepulse.drv/pulse.c b/dlls/winepulse.drv/pulse.c
index d259902c8b5..be540a4da2c 100644
--- a/dlls/winepulse.drv/pulse.c
+++ b/dlls/winepulse.drv/pulse.c
@@ -2489,7 +2489,6 @@ static NTSTATUS pulse_get_prop_value(void *args)
     struct list *list = (params->flow == eRender) ? &g_phys_speakers : &g_phys_sources;
     PhysDevice *dev;

-    params->result = S_OK;
     LIST_FOR_EACH_ENTRY(dev, list, PhysDevice, entry) {
         if (strcmp(params->device, dev->pulse_name))
             continue;
@@ -2501,18 +2500,19 @@ static NTSTATUS pulse_get_prop_value(void *args)
             case 0:   /* FormFactor */
                 params->value->vt = VT_UI4;
                 params->value->ulVal = dev->form;
+                params->result = S_OK;
                 return STATUS_SUCCESS;
             case 3:   /* PhysicalSpeakers */
                 if (!dev->channel_mask)
                     goto fail;
                 params->value->vt = VT_UI4;
                 params->value->ulVal = dev->channel_mask;
+                params->result = S_OK;
                 return STATUS_SUCCESS;
-            default:
-                params->result = E_NOTIMPL;
             }
         }

+        params->result = E_NOTIMPL;
         return STATUS_SUCCESS;
     }