[Bug 28737] shell32: invalid free in BrsFolderDlgProc in func_brsfolder in "make brsfolder.ok"?

http://bugs.winehq.org/show_bug.cgi?id=28737

--- Comment #6 from Michael Mc Donnell  2011-10-19 14:23:45 CDT ---
(In reply to comment #2)
> Created attachment 36987 [details]
> Patch that sets pointer to NULL after free to avoid double free
>
> This patch fixes the valgrind warning by setting the pointer to NULL in case of
> failure. I'm not going to send this to wine-patches because it really needs a
> test as Piotr Caban says.

On second thought. I don't think this patch will hurt anything because the
pointer it sets to NULL have already been set to a garbage value. A tests to
show if the value is changed in the first place would however be nice.

Any thoughts?