msacm32: Fix comment and remove unneeded if check (Coverity).

Thanks to Alex Villacís Lasso for comments.

msacm32: Fix comment and remove unneeded if check (Coverity).
c3bf8b88 · Kai Blin · Alexandre Julliard · 386bb905 · c3bf8b88
Commit c3bf8b88 authored Jan 06, 2007 by Kai Blin Committed by Alexandre Julliard Jan 06, 2007
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

driver.c dlls/msacm32/driver.c +4 -4

No files found.
--- a/dlls/msacm32/driver.c
+++ b/dlls/msacm32/driver.c
@@ -426,11 +426,11 @@ LRESULT WINAPI acmDriverMessage(HACMDRIVER had, UINT uMsg, LPARAM lParam1, LPARA
            }
            if (pAlias != NULL) {
-                unsigned int iStructSize = 16;
+                /* DRVCONFIGINFO is only 12 bytes long, but native msacm
-                /* This verification is required because DRVCONFIGINFO is 12 bytes
+                 * reports a 16-byte structure to codecs, so allocate 16 bytes,
-                   long, yet native msacm reports a 16-byte structure to codecs.
+                 * just to be on the safe side.
                 */
-                if (iStructSize < sizeof(DRVCONFIGINFO)) iStructSize = sizeof(DRVCONFIGINFO);
+                const unsigned int iStructSize = 16;
                pConfigInfo = HeapAlloc(MSACM_hHeap, 0, iStructSize);
                if (!pConfigInfo) {
                    ERR("OOM while supplying DRVCONFIGINFO for DRV_CONFIGURE, using NULL\n");