[cfinck] 68323: [LOCALSPL, WINPRINT] Bugfix: I want to dereference the pointer and then increment its value, not increment the pointer address.

Author: cfinck
Date: Tue Jun 30 16:02:26 2015
New Revision: 68323

URL: http://svn.reactos.org/svn/reactos?rev=68323&view=rev
Log:
[LOCALSPL, WINPRINT]
Bugfix: I want to dereference the pointer and then increment its value, not increment the pointer address.

Modified:
    branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c
    branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/jobs.c
    branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/printers.c

Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c
URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/reac...
==============================================================================
--- branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c	[iso-8859-1] (original)
+++ branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c	[iso-8859-1] Tue Jun 30 16:02:26 2015
@@ -131,7 +131,7 @@
         // Also calculate the offset in the output buffer of the pointer to this datatype string.
         *pCurrentOffset = *pcReturned * sizeof(DATATYPES_INFO_1W) + FIELD_OFFSET(DATATYPES_INFO_1W, pName);

-        *pcReturned++;
+        (*pcReturned)++;
         pCurrentOffset++;
     }

@@ -198,7 +198,7 @@
         goto Cleanup;
     }

-    // Create a new WINPRINT_HANDLE structure. and fill the relevant fields.
+    // Create a new WINPRINT_HANDLE structure and fill the relevant fields.
     pHandle = DllAllocSplMem(sizeof(WINPRINT_HANDLE));

// Check what datatype was given.

Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/jobs.c
URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/reac...
==============================================================================
--- branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/jobs.c	[iso-8859-1] (original)
+++ branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/jobs.c	[iso-8859-1] Tue Jun 30 16:02:26 2015
@@ -943,7 +943,7 @@
             _LocalGetJobLevel2(pPrinterHandle, pJob, NULL, NULL, 0, pcbNeeded);

// We stop either when there are no more jobs in the list or when the caller didn't request more, whatever comes first.
-        *pcReturned++;
+        (*pcReturned)++;
         pNode = pNode->Next[0];
     }

@@ -976,7 +976,7 @@
             goto Cleanup;

// We stop either when there are no more jobs in the list or when the caller didn't request more, whatever comes first.
-        *pcReturned++;
+        (*pcReturned)++;
         pNode = pNode->Next[0];
     }

Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/printers.c
URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/reac...
==============================================================================
--- branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/printers.c	[iso-8859-1] (original)
+++ branches/colins-printing-for-freedom/reactos/win32ss/printing/providers/localspl/printers.c	[iso-8859-1] Tue Jun 30 16:02:26 2015
@@ -347,7 +347,7 @@
         cbDescription = cchComputerName * sizeof(WCHAR) + cbName + cbComment + sizeof(WCHAR);

*pcbNeeded += sizeof(PRINTER_INFO_1W) + cchComputerName * sizeof(WCHAR) + cbName + cbComment + cbDescription;
-        *pcReturned++;
+        (*pcReturned)++;
     }

// Check if the supplied buffer is large enough.