Updated C6333

Updated to new format, modified wording, changed whitespace to match my other PRs

Author: MugBergerFries

docs/code-quality/c6333.md

@@ -1,21 +1,25 @@
 ---
 description: "Learn more about: C6333"
 title: C6333
-ms.date: 11/04/2016
+ms.date: 08/25/2022
 ms.topic: reference
-f1_keywords: ["C6333"]
+f1_keywords: ["C6333", "VIRTUALFREEINVALIDPARAM3", "__WARNING_VIRTUALFREEINVALIDPARAM3"]
 helpviewer_keywords: ["C6333"]
 ms.assetid: 4b8fa4b2-a3a0-4d00-bec7-76686b66fcf9
 ---
-# C6333
+# Warning C6333
 
-> warning C6333: Invalid parameter: passing MEM_RELEASE and a non-zero dwSize parameter to \<function> is not allowed. This results in the failure of this call
+> Invalid parameter: passing MEM_RELEASE and a non-zero dwSize parameter to '\**function_name*' is not allowed. This results in the failure of this call
 
-This warning indicates an invalid parameter is being passed to VirtualFree or VirtualFreeEx. Both of these functions reject a dwFreeType of MEM_RELEASE with a non-zero value of dwSize. When passing MEM_RELEASE, the dwSize parameter must be zero. Also, make sure that the return value of this function is not ignored.
+## Remarks
+
+Both `VirtualFree` and `VirtualFreeEx` reject a `dwFreeType` of `MEM_RELEASE` with a non-zero value of `dwSize`. When passing `MEM_RELEASE`, the `dwSize` parameter must be zero.
+
+Code analysis name: VIRTUALFREEINVALIDPARAM3
 
 ## Example
 
-The following sample code generates this warning:
+The following code sample generates this warning:
 
 ```cpp
 #include <windows.h>
@@ -24,36 +28,36 @@ The following sample code generates this warning:
 DWORD dwPages = 0;  // count of pages
 DWORD dwPageSize;   // page size
 
-VOID f( VOID )
+VOID f(VOID)
 {
-  LPVOID lpvBase;            // base address of the test memory
-  BOOL bSuccess;
-  SYSTEM_INFO sSysInfo;      // system information
-
-  GetSystemInfo( &sSysInfo );
-  dwPageSize = sSysInfo.dwPageSize;
-
-  // Reserve pages in the process's virtual address space
-  lpvBase = VirtualAlloc(
-                         NULL,                // system selects address
-                         PAGELIMIT*dwPageSize,// size of allocation
-                         MEM_RESERVE,
-                         PAGE_NOACCESS );
-  if (lpvBase)
-  {
-    // code to access memory
-  }
-  else
-  {
-    return;
-  }
-
-  bSuccess = VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_RELEASE);
-  //code...
+    LPVOID lpvBase;            // base address of the test memory
+    BOOL bSuccess;
+    SYSTEM_INFO sSysInfo;      // system information
+
+    GetSystemInfo(&sSysInfo);
+    dwPageSize = sSysInfo.dwPageSize;
+
+    // Reserve pages in the process's virtual address space
+    lpvBase = VirtualAlloc(
+                           NULL,                // system selects address
+                           PAGELIMIT*dwPageSize,// size of allocation
+                           MEM_RESERVE,
+                           PAGE_NOACCESS);
+    if (lpvBase)
+    {
+        // code to access memory
+    }
+    else
+    {
+        return;
+    }
+
+    bSuccess = VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_RELEASE);
+    //code...
 }
 ```
 
-To correct this warning, use the following sample code:
+To correct this warning, ensure that the value of `dwSize` is 0 in the call to `VirtualFree`:
 
 ```cpp
 #include <windows.h>
@@ -62,37 +66,37 @@ To correct this warning, use the following sample code:
 DWORD dwPages = 0;  // count of pages
 DWORD dwPageSize;   // page size
 
-VOID f( VOID )
+VOID f(VOID)
 {
-  LPVOID lpvBase;            // base address of the test memory
-  BOOL bSuccess;
-  SYSTEM_INFO sSysInfo;      // system information
-
-  GetSystemInfo( &sSysInfo );
-  dwPageSize = sSysInfo.dwPageSize;
-
-  // Reserve pages in the process's virtual address space
-  lpvBase = VirtualAlloc(
-                         NULL,                // system selects address
-                         PAGELIMIT*dwPageSize,// size of allocation
-                         MEM_RESERVE,
-                         PAGE_NOACCESS );
-  if (lpvBase)
-  {
-    // code to access memory
-  }
-  else
-  {
-    return;
-  }
-  bSuccess = VirtualFree(lpvBase, 0, MEM_RELEASE );
-
-  //  VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_DECOMMIT);
-  // code...
+    LPVOID lpvBase;            // base address of the test memory
+    BOOL bSuccess;
+    SYSTEM_INFO sSysInfo;      // system information
+
+    GetSystemInfo(&sSysInfo);
+    dwPageSize = sSysInfo.dwPageSize;
+
+    // Reserve pages in the process's virtual address space
+    lpvBase = VirtualAlloc(
+                           NULL,                  // system selects address
+                           PAGELIMIT*dwPageSize,  // size of allocation
+                           MEM_RESERVE,
+                           PAGE_NOACCESS);
+    if (lpvBase)
+    {
+        // code to access memory
+    }
+    else
+    {
+        return;
+    }
+    bSuccess = VirtualFree(lpvBase, 0, MEM_RELEASE);
+
+    //  VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_DECOMMIT);
+    // code...
 }
 ```
 
-You can also use VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_DECOMMIT); call to decommit pages, and later release them using MEM_RELEASE flag.
+You can also use `VirtualFree(lpvBase, PAGELIMIT * dwPageSize, MEM_DECOMMIT);` call to decommit pages, and later release them using `MEM_RELEASE` flag.
 
 ## See also