On 11.06.2013 22:08, Christian Costa wrote:

Fixes bug 26598.
---
��� dlls/d3dx9_36/effect.c ��� ��� ��� | ��� ���4 ++++
��� dlls/d3dx9_36/tests/effect.c | ��� 17 +++++++++++++++++
��� 2 files changed, 21 insertions(+)

diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c
index 1924c07..bab4560 100644
--- a/dlls/d3dx9_36/effect.c
+++ b/dlls/d3dx9_36/effect.c
@@ -5792,6 +5792,10 @@ HRESULT WINAPI D3DXCreateEffectEx(struct IDirect3DDevice9 *device, const void *s

��� ��� ��� *effect = &object->ID3DXEffect_iface;

+ ��� ���/* Must be set to NULL if no compilation error */
+ ��� ���if (compilation_errors)
+ ��� ��� ��� ���*compilation_errors = NULL;
+

No, this is wrong! Your test case doesn't cover all cases.