[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

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