Changeset 87042 in vbox for trunk/src/VBox/Runtime/r3

Timestamp:

Dec 4, 2020 12:10:36 PM (4 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

141732

Message:

Shared Clipboard/Transfers: More cleanup / documentation work. bugref:9874

File:

• trunk/src/VBox/Runtime/r3/http-server.cpp (modified) (5 diffs)

trunk/src/VBox/Runtime/r3/http-server.cpp

@@ -328 +328 @@
 
 /**
+ * Initializes a HTTP body.
+ *
+ * @param   pBody               Body to initialize.
+ * @param   cbSize              Size of body (in bytes) to allocate. Optional and can be 0.
+ */
+static int rtHttpServerBodyInit(PRTHTTPBODY pBody, size_t cbSize)
+{
+    if (cbSize)
+    {
+        pBody->pvBody      = RTMemAlloc(cbSize);
+        AssertPtrReturn(pBody->pvBody, VERR_NO_MEMORY);
+        pBody->cbBodyAlloc = cbSize;
+    }
+    else
+    {
+        pBody->pvBody      = NULL;
+        pBody->cbBodyAlloc = 0;
+    }
+
+    pBody->cbBodyUsed = 0;
+    pBody->offBody    = 0;
+
+    return VINF_SUCCESS;
+}
+
+/**
+ * Destroys a HTTP body.
+ *
+ * @param   pBody               Body to destroy.
+ */
+static void rtHttpServerBodyDestroy(PRTHTTPBODY pBody)
+{
+    if (!pBody)
+        return;
+
+    if (pBody->pvBody)
+    {
+        Assert(pBody->cbBodyAlloc);
+
+        RTMemFree(pBody->pvBody);
+        pBody->pvBody = NULL;
+    }
+
+    pBody->cbBodyAlloc  = 0;
+    pBody->cbBodyUsed   = 0;
+    pBody->offBody      = 0;
+}
+
+/**
  * Allocates and initializes a new client request.
  *
@@ -341 +390 @@
     AssertRC(rc2);
 
+    rc2 = rtHttpServerBodyInit(&pReq->Body, 0 /* cbSize */);
+    AssertRC(rc2);
+
     return pReq;
 }
@@ -356 +408 @@
     RTHttpHeaderListDestroy(pReq->hHdrLst);
 
-    RTMemFree(pReq->pvBody);
-    pReq->pvBody = NULL;
+    rtHttpServerBodyDestroy(&pReq->Body);
 
     RTMemFree(pReq);
@@ -376 +427 @@
     AssertRCReturn(rc, rc);
 
-    if (cbBody)
-    {
-        pResp->pvBody      = RTMemAlloc(cbBody);
-        AssertPtrReturn(pResp->pvBody, VERR_NO_MEMORY);
-        pResp->cbBodyAlloc = cbBody;
-    }
-    else
-    {
-        pResp->cbBodyAlloc = 0;
-    }
-
-    pResp->cbBodyUsed = 0;
+    rc = rtHttpServerBodyInit(&pResp->Body, cbBody);
 
     return rc;
@@ -417 +457 @@
     RTHttpHeaderListDestroy(pResp->hHdrLst);
 
-    if (pResp->pvBody)
-    {
-        Assert(pResp->cbBodyAlloc);
-
-        RTMemFree(pResp->pvBody);
-        pResp->pvBody = NULL;
-    }
-
-    pResp->cbBodyAlloc = 0;
-    pResp->cbBodyUsed  = 0;
+    rtHttpServerBodyDestroy(&pResp->Body);
 }