[PATCH v2 0/1] MR3918: msado15: Recordset::Open - Correctly update count/index values

[Alistair Leslie-Hughes (＠alesliehughes)]

ADO can return a RecordCount of -1.

--
  v2: msado15: Recordset::Open - Correctly update count/index values
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3918

[PATCH v2 1/1] msado15: Recordset::Open - Correctly update count/index values

[Alistair Leslie-Hughes]

From: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com>

ADO can return a RecordCount of -1.
---
 dlls/msado15/recordset.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/msado15/recordset.c b/dlls/msado15/recordset.c
index 84db7fc581d..a70fc2b33ac 100644
--- a/dlls/msado15/recordset.c
+++ b/dlls/msado15/recordset.c
@@ -2104,8 +2104,8 @@ static HRESULT WINAPI recordset_Open( _Recordset *iface, VARIANT source, VARIANT
         return hr;
     }
 
-    recordset->count = affected;
-    recordset->index = affected ? 0 : -1;
+    recordset->count = affected > 0 ? affected : 0;
+    recordset->index = affected > 0 ? 0 : -1;
 
     /*
      * We can safely just return with an empty recordset here
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/3918