Re: [PATCH v2 0/1] MR3016: dbghelp: Allocate real_path with the CRT and copy it to the module heap.

[eric pouech (＠epo)]

eric pouech (@epo) commented about dlls/dbghelp/pe_module.c:
>      if (name) lstrcpyW(loaded_name, name);
>  }
>  if (!(modfmt = HeapAlloc(GetProcessHeap(), 0, sizeof(struct module_format) + sizeof(struct pe_module_info))))
> -
>    return NULL;
> -
>    goto done;

I'd rather avoid the goto, and most importantly you don't close the file handle. perhaps
```
modfmt = HeapAlloc(GetProcessHeap(), 0, sizeof(struct module_format) + sizeof(struct pe_module_info));
if (modfmt)
{
...
}
if (!module) HeapFree(...)
```