crypt32: Fix order-of-evaluation dependence

Michael Karcher wine at mkarcher.dialup.fu-berlin.de
Sun May 25 10:38:54 CDT 2008


This patch should go into wine 1.0. Debians gcc 4.2.3-5 compiles it in
the non-intended way, so the NotAfter field gets the NotBefore time.

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

diff --git a/dlls/crypt32/encode.c b/dlls/crypt32/encode.c
index a274f44..80f30ae 100644
--- a/dlls/crypt32/encode.c
+++ b/dlls/crypt32/encode.c
@@ -346,8 +346,8 @@ static BOOL WINAPI CRYPT_AsnEncodeValidity(DWORD dwCertEncodingType,
     /* This has two filetimes in a row, a NotBefore and a NotAfter */
     const FILETIME *timePtr = (const FILETIME *)pvStructInfo;
     struct AsnEncodeSequenceItem items[] = {
-     { timePtr++, CRYPT_AsnEncodeChoiceOfTime, 0 },
-     { timePtr,   CRYPT_AsnEncodeChoiceOfTime, 0 },
+     { timePtr,     CRYPT_AsnEncodeChoiceOfTime, 0 },
+     { timePtr + 1, CRYPT_AsnEncodeChoiceOfTime, 0 },
     };
 
     ret = CRYPT_AsnEncodeSequence(dwCertEncodingType, items, 
-- 
1.5.5.1




More information about the wine-patches mailing list