Michael Stefaniuc : secur32: Use offsetof() instead of pointer substraction.

Alexandre Julliard julliard at winehq.org
Tue Feb 19 15:29:35 CST 2019


Module: wine
Branch: master
Commit: 8b343fd55b25f52070fab36f32c19d64d2641b8e
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=8b343fd55b25f52070fab36f32c19d64d2641b8e

Author: Michael Stefaniuc <mstefani at winehq.org>
Date:   Tue Feb 19 20:44:31 2019 +0100

secur32: Use offsetof() instead of pointer substraction.

Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/secur32/secur32.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/secur32/secur32.c b/dlls/secur32/secur32.c
index b3ab80a..8288286 100644
--- a/dlls/secur32/secur32.c
+++ b/dlls/secur32/secur32.c
@@ -247,8 +247,8 @@ static void _makeFnTableA(PSecurityFunctionTableA fnTableA,
              * implemented (yikes)
              */
             size_t tableSize = inFnTableA->dwVersion == 1 ?
-             (const BYTE *)&inFnTableA->SetContextAttributesA -
-             (const BYTE *)inFnTableA : sizeof(SecurityFunctionTableA);
+             offsetof(SecurityFunctionTableA, SetContextAttributesA) :
+             sizeof(SecurityFunctionTableA);
 
             memcpy(fnTableA, inFnTableA, tableSize);
             /* override this, since we can do it internally anyway */
@@ -318,8 +318,8 @@ static void _makeFnTableW(PSecurityFunctionTableW fnTableW,
              * implemented (yikes)
              */
             size_t tableSize = inFnTableW->dwVersion == 1 ?
-             (const BYTE *)&inFnTableW->SetContextAttributesW -
-             (const BYTE *)inFnTableW : sizeof(SecurityFunctionTableW);
+             offsetof(SecurityFunctionTableW, SetContextAttributesW) :
+             sizeof(SecurityFunctionTableW);
 
             memcpy(fnTableW, inFnTableW, tableSize);
             /* override this, since we can do it internally anyway */




More information about the wine-cvs mailing list