The BOM doesn't need a DWORD sized variable, only a WCHAR one.

Don't increment the szFile pointer since we removed the BOM earlier and it will cause the first real character of the INI file to be skipped.

The BOM doesn't need a DWORD sized variable, only a WCHAR one.
eeb0062f · Robert Shearman · Alexandre Julliard · f42fac55 · eeb0062f
Commit eeb0062f authored Nov 17, 2005 by Robert Shearman Committed by Alexandre Julliard Nov 17, 2005
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

profile.c dlls/kernel/profile.c +3 -3

No files found.
--- a/dlls/kernel/profile.c
+++ b/dlls/kernel/profile.c
@@ -132,7 +132,7 @@ static inline void PROFILE_ByteSwapShortBuffer(WCHAR * buffer, int len)
 static inline void PROFILE_WriteMarker(HANDLE hFile, ENCODING encoding)
 {
    DWORD dwBytesWritten;
-    DWORD bom;
+    WCHAR bom;
    switch (encoding)
    {
    case ENCODING_ANSI:
@@ -373,12 +373,12 @@ static PROFILESECTION *PROFILE_Load(HANDLE hFile, ENCODING * pEncoding)
        break;
    case ENCODING_UTF16LE:
        TRACE("UTF16 Little Endian encoding\n");
-        szFile = (WCHAR *)pBuffer + 1;
+        szFile = (WCHAR *)pBuffer;
        szEnd = (WCHAR *)((char *)pBuffer + dwFileSize);
        break;
    case ENCODING_UTF16BE:
        TRACE("UTF16 Big Endian encoding\n");
-        szFile = (WCHAR *)pBuffer + 1;
+        szFile = (WCHAR *)pBuffer;
        szEnd = (WCHAR *)((char *)pBuffer + dwFileSize);
        PROFILE_ByteSwapShortBuffer(szFile, dwFileSize / sizeof(WCHAR));
        break;