- Revert so call "Fixes" introduced in 18941 because they are bugs which I had to fix in 18002 and adding back bugs isn't really a good idea.

Modified: trunk/reactos/lib/rtl/unicode.c

--- trunk/reactos/lib/rtl/unicode.c	2005-11-01 22:45:35 UTC (rev 18941)
+++ trunk/reactos/lib/rtl/unicode.c	2005-11-01 23:37:26 UTC (rev 18942)
@@ -357,6 +357,7 @@

     if (AnsiString->Buffer)
     {
         RtlpFreeStringMemory(AnsiString->Buffer, TAG_ASTR);

+        RtlZeroMemory(AnsiString, sizeof(ANSI_STRING));

     }
 }
 
@@ -369,10 +370,7 @@

 {
    PAGED_CODE_RTL();
 

-   if (OemString->Buffer)
-   {
-       RtlpFreeStringMemory(OemString->Buffer, TAG_OSTR);
-   }

+   if (OemString->Buffer) RtlpFreeStringMemory(OemString->Buffer, TAG_OSTR);

 }
 
 /*
@@ -387,6 +385,7 @@

     if (UnicodeString->Buffer)
     {
         RtlpFreeStringMemory(UnicodeString->Buffer, TAG_ASTR);

+        RtlZeroMemory(UnicodeString, sizeof(UNICODE_STRING));

     }
 }
 
@@ -485,7 +484,7 @@

     if(SourceString)
     {
         DestSize = wcslen(SourceString) * sizeof(WCHAR);

-        if (DestSize > 0xFFFE) return STATUS_NAME_TOO_LONG;

+        if (DestSize >= 0xFFFC) return STATUS_NAME_TOO_LONG;

         DestinationString->Length = (USHORT)DestSize;
         DestinationString->MaximumLength = (USHORT)DestSize + sizeof(WCHAR);
     }
@@ -1944,8 +1943,6 @@

     PAGED_CODE_RTL();
 
     Length = (wcslen(Source) + 1) * sizeof(WCHAR);

-    if (Length > 0xFFFE) return FALSE;
-

     UniDest->Buffer = RtlpAllocateStringMemory(Length, TAG_USTR);
 
     if (UniDest->Buffer == NULL) return FALSE;