Alexandre Julliard : kernel32: Avoid using memrchrW().
Alexandre Julliard
julliard at winehq.org
Wed Apr 1 15:50:58 CDT 2020
Module: wine
Branch: master
Commit: ce30db504bb61e026736ac0b24ce00ba587044de
URL: https://source.winehq.org/git/wine.git/?a=commit;h=ce30db504bb61e026736ac0b24ce00ba587044de
Author: Alexandre Julliard <julliard at winehq.org>
Date: Wed Apr 1 10:28:57 2020 +0200
kernel32: Avoid using memrchrW().
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/kernel32/profile.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/dlls/kernel32/profile.c b/dlls/kernel32/profile.c
index e4bd861b14..c7382fd7b3 100644
--- a/dlls/kernel32/profile.c
+++ b/dlls/kernel32/profile.c
@@ -418,8 +418,8 @@ static PROFILESECTION *PROFILE_Load(HANDLE hFile, ENCODING * pEncoding)
if (*szLineStart == '[') /* section start */
{
- const WCHAR * szSectionEnd;
- if (!(szSectionEnd = memrchrW( szLineStart, ']', szLineEnd - szLineStart )))
+ for (len = szLineEnd - szLineStart; len > 0; len--) if (szLineStart[len - 1] == ']') break;
+ if (!len)
{
WARN("Invalid section header at line %d: %s\n",
line, debugstr_wn(szLineStart, (int)(szLineEnd - szLineStart)) );
@@ -427,7 +427,7 @@ static PROFILESECTION *PROFILE_Load(HANDLE hFile, ENCODING * pEncoding)
else
{
szLineStart++;
- len = (int)(szSectionEnd - szLineStart);
+ len -= 2;
/* no need to allocate +1 for NULL terminating character as
* already included in structure */
if (!(section = HeapAlloc( GetProcessHeap(), 0, sizeof(*section) + len * sizeof(WCHAR) )))
More information about the wine-cvs
mailing list