Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/effect.c:
goto fail; }
skip_constants_buffer = HeapAlloc(GetProcessHeap(), 0,
sizeof(*skip_constants_buffer) * (strlen(skip_constants_string) + 1));
skip_constants_buffer = malloc(sizeof(*skip_constants_buffer) * (strlen(skip_constants_string) + 1));
No big deal, but for this one there is really no need to multiply by sizeof(char).