Fix test case and comment

Author: guhetier

src/core/recv_buffer.c

@@ -273,6 +273,7 @@ QuicRecvBufferInitialize(
 {
     CXPLAT_DBG_ASSERT(AllocBufferLength != 0 || RecvMode == QUIC_RECV_BUF_MODE_APP_OWNED);
     CXPLAT_DBG_ASSERT(VirtualBufferLength != 0 || RecvMode == QUIC_RECV_BUF_MODE_APP_OWNED);
+    CXPLAT_DBG_ASSERT(PreallocatedChunk == NULL || RecvMode != QUIC_RECV_BUF_MODE_APP_OWNED);
     CXPLAT_DBG_ASSERT((AllocBufferLength & (AllocBufferLength - 1)) == 0);     // Power of 2
     CXPLAT_DBG_ASSERT((VirtualBufferLength & (VirtualBufferLength - 1)) == 0); // Power of 2
     CXPLAT_DBG_ASSERT(AllocBufferLength <= VirtualBufferLength);

src/core/recv_buffer.h

@@ -105,7 +105,8 @@ typedef struct QUIC_RECV_BUFFER {
 //
 // Initialize a QUIC_RECV_BUFFER.
 // Can only fail if PreallocatedChunk == NULL && RecvMode != QUIC_RECV_BUF_MODE_APP_OWNED.
-// PreallocatedChunk is owned by the caller and must be freed afte the buffer is uninitialized.
+// PreallocatedChunk ownership is given to the receive buffer.
+// PreallocatedChunk must be null if RecvMode == QUIC_RECV_BUF_MODE_APP_OWNED.
 //
 _IRQL_requires_max_(DISPATCH_LEVEL)
 QUIC_STATUS

src/core/unittest/RecvBufferTest.cpp

@@ -312,6 +312,11 @@ TEST_P(WithMode, Alloc)
 TEST_P(WithMode, AllocWithChunk)
 {
     RecvBuffer RecvBuf;
+    const auto Mode = GetParam();
+    if (Mode == QUIC_RECV_BUF_MODE_APP_OWNED) {
+        // App-owned mode doesn't support preallocated chunks
+        return;
+    }
     ASSERT_EQ(QUIC_STATUS_SUCCESS, RecvBuf.Initialize(GetParam(), true));
 }