msvcrt: move error condition before memory allocation (found by Smatch) [2/3, resend]

[Lionel_Debroux]

_fullpath leaks some heap memory in an error paths. Found in Michael
Stefaniuc's list of Wine potential bugs detected by Smatch.
Fixed by moving the error treatment before the allocation, since the
error treatment doesn't fire up if there was a memory allocation (due to
size = MAX_PATH; assignment).

Is there anything wrong with this patch ?

2007-09-24  Lionel Debroux <lionel_debroux at yahoo.fr>
        * dlls/msvcrt/dir.c:
        msvcrt: move error condition before memory allocation (found by
Smatch).


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-msvcrt-move-error-condition-before-memory-allocatio.patch
Type: text/x-patch
Size: 1138 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20070927/12347e39/attachment.bin