Commit d2ab8296 authored by Dmitry Timoshkov's avatar Dmitry Timoshkov Committed by Alexandre Julliard

adsldp: Improve IDirectorySearch::GetNextRow() end of entries detection logic.

parent 7758d7a1
...@@ -1211,11 +1211,9 @@ static HRESULT WINAPI search_GetFirstRow(IDirectorySearch *iface, ADS_SEARCH_HAN ...@@ -1211,11 +1211,9 @@ static HRESULT WINAPI search_GetFirstRow(IDirectorySearch *iface, ADS_SEARCH_HAN
if (!res) return E_ADS_BAD_PARAMETER; if (!res) return E_ADS_BAD_PARAMETER;
ldap_ctx->count = ldap_count_entries(ldap->ld, ldap_ctx->res); ldap_ctx->entry = NULL;
ldap_ctx->pos = 0;
ldap_ctx->entry = ldap_first_entry(ldap->ld, ldap_ctx->res);
return ldap_ctx->entry ? S_OK : S_ADS_NOMORE_ROWS; return IDirectorySearch_GetNextRow(iface, res);
} }
static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HANDLE res) static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HANDLE res)
...@@ -1233,6 +1231,10 @@ static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HAND ...@@ -1233,6 +1231,10 @@ static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HAND
{ {
ldap_ctx->count = ldap_count_entries(ldap->ld, ldap_ctx->res); ldap_ctx->count = ldap_count_entries(ldap->ld, ldap_ctx->res);
ldap_ctx->pos = 0; ldap_ctx->pos = 0;
if (ldap_ctx->pos >= ldap_ctx->count)
return S_ADS_NOMORE_ROWS;
ldap_ctx->entry = ldap_first_entry(ldap->ld, ldap_ctx->res); ldap_ctx->entry = ldap_first_entry(ldap->ld, ldap_ctx->res);
} }
else else
...@@ -1241,10 +1243,14 @@ static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HAND ...@@ -1241,10 +1243,14 @@ static HRESULT WINAPI search_GetNextRow(IDirectorySearch *iface, ADS_SEARCH_HAND
return S_ADS_NOMORE_ROWS; return S_ADS_NOMORE_ROWS;
ldap_ctx->entry = ldap_next_entry(ldap->ld, ldap_ctx->res); ldap_ctx->entry = ldap_next_entry(ldap->ld, ldap_ctx->res);
ldap_ctx->pos++;
} }
return ldap_ctx->entry ? S_OK : S_ADS_NOMORE_ROWS; if (!ldap_ctx->entry)
return S_ADS_NOMORE_ROWS;
ldap_ctx->pos++;
return S_OK;
} }
static HRESULT WINAPI search_GetPreviousRow(IDirectorySearch *iface, ADS_SEARCH_HANDLE res) static HRESULT WINAPI search_GetPreviousRow(IDirectorySearch *iface, ADS_SEARCH_HANDLE res)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment