[ros-diffs] [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/rea…
==============================================================================
---
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/rea…
==============================================================================
---
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/rea…
==============================================================================
---
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.