Rob Shearman : advapi32: Set the right pointer so that StringAcl is updated correct in ParseAceStringRights .

Alexandre Julliard julliard at wine.codeweavers.com
Thu Jul 19 07:59:41 CDT 2007


Module: wine
Branch: master
Commit: ccfa9ad8df1c866cfb0267568f6b1254c5049439
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ccfa9ad8df1c866cfb0267568f6b1254c5049439

Author: Rob Shearman <rob at codeweavers.com>
Date:   Wed Jul 18 18:12:43 2007 +0100

advapi32: Set the right pointer so that StringAcl is updated correct in ParseAceStringRights.

The number of allowed hex digits should be 8, not 6.

---

 dlls/advapi32/security.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/advapi32/security.c b/dlls/advapi32/security.c
index aa51515..7de2ab8 100644
--- a/dlls/advapi32/security.c
+++ b/dlls/advapi32/security.c
@@ -3125,10 +3125,10 @@ static DWORD ParseAceStringRights(LPCWSTR* StringAcl)
 	while (*p && *p != ';')
             p++;
 
-	if (p - szAcl <= 8)
+	if (p - szAcl <= 10 /* 8 hex digits + "0x" */ )
 	{
 	    rights = strtoulW(szAcl, NULL, 16);
-	    *StringAcl = p;
+	    szAcl = p;
 	}
 	else
             WARN("Invalid rights string format: %s\n", debugstr_wn(szAcl, p - szAcl));




More information about the wine-cvs mailing list