commit 1fad872961499c48c2b33851636f8408d67321cc
author Maarten Lankhorst <m.b.lankhorst@gmail.com> Mon Apr 28 09:26:20 2008 -0700
committer Alexandre Julliard <julliard@winehq.org> Tue Apr 29 12:45:17 2008 +0200
tree 25a0796be27ec5956e3471d2b8d80c696722679e
parent 928914c81ebbbb21ff66e3b044036bf7a4172d08

quartz: Fix bug in memallocator with test.

dlls/quartz/memallocator.c

diff --git a/dlls/quartz/memallocator.c b/dlls/quartz/memallocator.c
index cd87a34..12effbb 100644
--- a/dlls/quartz/memallocator.c
+++ b/dlls/quartz/memallocator.c
@@ -237,13 +237,13 @@
     {
         if (!This->pProps)
             hr = VFW_E_SIZENOTSET;
-        else if (This->bCommitted)
-            hr = S_OK;
-        else if (This->bDecommitQueued)
+        else if (This->bDecommitQueued && This->bCommitted)
         {
             This->bDecommitQueued = FALSE;
             hr = S_OK;
         }
+        else if (This->bCommitted)
+            hr = S_OK;
         else
         {
             if (!(This->hSemWaiting = CreateSemaphoreW(NULL, This->pProps->cBuffers, This->pProps->cBuffers, NULL)))

dlls/quartz/tests/memallocator.c

diff --git a/dlls/quartz/tests/memallocator.c b/dlls/quartz/tests/memallocator.c
index 0a01db9..b0638e8 100644
--- a/dlls/quartz/tests/memallocator.c
+++ b/dlls/quartz/tests/memallocator.c
@@ -37,13 +37,15 @@
 
     if (hr == S_OK)
     {
-	ALLOCATOR_PROPERTIES RequestedProps;
-	ALLOCATOR_PROPERTIES ActualProps;
+        ALLOCATOR_PROPERTIES RequestedProps;
+        ALLOCATOR_PROPERTIES ActualProps;
 
-	RequestedProps.cBuffers = 1;
-	RequestedProps.cbBuffer = 65536;
-	RequestedProps.cbAlign = 1;
-	RequestedProps.cbPrefix = 0;
+        IMediaSample *sample = NULL, *sample2 = NULL;
+
+        RequestedProps.cBuffers = 2;
+        RequestedProps.cbBuffer = 65536;
+        RequestedProps.cbAlign = 1;
+        RequestedProps.cbPrefix = 0;
 
 	hr = IMemAllocator_SetProperties(pMemAllocator, &RequestedProps, &ActualProps);
 	ok(hr==S_OK, "SetProperties returned: %x\n", hr);
@@ -53,12 +55,30 @@
 	hr = IMemAllocator_Commit(pMemAllocator);
 	ok(hr==S_OK, "Commit returned: %x\n", hr);
 
+        hr = IMemAllocator_GetBuffer(pMemAllocator, &sample, NULL, NULL, 0);
+        ok(hr==S_OK, "Could not get a buffer: %x\n", hr);
+
 	hr = IMemAllocator_Decommit(pMemAllocator);
 	ok(hr==S_OK, "Decommit returned: %x\n", hr);
 	hr = IMemAllocator_Decommit(pMemAllocator);
 	ok(hr==S_OK, "Cecommit returned: %x\n", hr);
 
-	IMemAllocator_Release(pMemAllocator);
+        /* Decommit and recommit while holding a sample */
+        if (sample)
+        {
+            hr = IMemAllocator_Commit(pMemAllocator);
+            ok(hr==S_OK, "Commit returned: %x\n", hr);
+
+            hr = IMemAllocator_GetBuffer(pMemAllocator, &sample2, NULL, NULL, 0);
+            ok(hr==S_OK, "Could not get a buffer: %x\n", hr);
+            IUnknown_Release(sample);
+            if (sample2)
+                IUnknown_Release(sample2);
+
+            hr = IMemAllocator_Decommit(pMemAllocator);
+            ok(hr==S_OK, "Cecommit returned: %x\n", hr);
+        }
+        IMemAllocator_Release(pMemAllocator);
     }
 }