From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Sat, 21 Jul 2012 15:57:40 +0000 (-0400)
Subject: Windows: Freeing SeQueryInformationToken allocations
X-Git-Tag: upstream/1.8.0_pre1^2~2196
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=8a11435eed2d3d1d1aef0ebb92822b34299b2e4d;p=packages%2Fo%2Fopenafs.git

Windows: Freeing SeQueryInformationToken allocations

Memory allocated by SeQueryInformationToken() must be freed with
ExFreePool() and not AFSExFreePool().

Change-Id: Iece2c37d5dc524204cbf3d8d6869148e1a0ce88e
Reviewed-on: http://gerrit.openafs.org/7806
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsrdr/kernel/fs/AFSGeneric.cpp b/src/WINNT/afsrdr/kernel/fs/AFSGeneric.cpp
index e5260775e..7679302a7 100644
--- a/src/WINNT/afsrdr/kernel/fs/AFSGeneric.cpp
+++ b/src/WINNT/afsrdr/kernel/fs/AFSGeneric.cpp
@@ -1787,7 +1787,7 @@ try_exit:
 
         if( pTokenInfo != NULL)
         {
-            AFSExFreePool( pTokenInfo);
+            ExFreePool( pTokenInfo);    // Allocated by SeQueryInformationToken
         }
     }
 
diff --git a/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp b/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp
index ba6ba809a..4baec3bc1 100644
--- a/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp
+++ b/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp
@@ -7614,7 +7614,7 @@ try_exit:
         if( pTokenInfo != NULL)
         {
 
-            AFSExFreePool( pTokenInfo);
+            ExFreePool( pTokenInfo);    // Allocated by SeQueryInformationToken
         }
     }