Alistair Leslie-Hughes : advapi32: Correct off by one in LookupPrivilegeValueW.

Alexandre Julliard julliard at winehq.org
Mon Nov 10 07:44:08 CST 2008


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

Author: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
Date:   Mon Nov 10 19:47:29 2008 +1100

advapi32: Correct off by one in LookupPrivilegeValueW.

---

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

diff --git a/dlls/advapi32/security.c b/dlls/advapi32/security.c
index 67c410f..8229377 100644
--- a/dlls/advapi32/security.c
+++ b/dlls/advapi32/security.c
@@ -1679,7 +1679,7 @@ LookupPrivilegeValueW( LPCWSTR lpSystemName, LPCWSTR lpName, PLUID lpLuid )
         SetLastError(ERROR_NO_SUCH_PRIVILEGE);
         return FALSE;
     }
-    for( i=SE_MIN_WELL_KNOWN_PRIVILEGE; i<SE_MAX_WELL_KNOWN_PRIVILEGE; i++ )
+    for( i=SE_MIN_WELL_KNOWN_PRIVILEGE; i<=SE_MAX_WELL_KNOWN_PRIVILEGE; i++ )
     {
         if( !WellKnownPrivNames[i] )
             continue;
diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
index 2d58946..3d02777 100644
--- a/dlls/advapi32/tests/security.c
+++ b/dlls/advapi32/tests/security.c
@@ -579,7 +579,7 @@ static void test_lookupPrivilegeName(void)
      "SeCreateTokenPrivilege (got %d, expected %d)\n", cchName,
      (int)strlen("SeCreateTokenPrivilege"));
     /* check known values */
-    for (i = SE_MIN_WELL_KNOWN_PRIVILEGE; i < SE_MAX_WELL_KNOWN_PRIVILEGE; i++)
+    for (i = SE_MIN_WELL_KNOWN_PRIVILEGE; i <= SE_MAX_WELL_KNOWN_PRIVILEGE; i++)
     {
         luid.LowPart = i;
         cchName = sizeof(buf);




More information about the wine-cvs mailing list