setupapi: Don't copy into a NULL PathBuffer, and remove an invalid check for size < MAX_PATH (Coverity 888).

Ricardo Filipe ricardojdfilipe at gmail.com
Sun Mar 22 18:04:11 CDT 2009


my next task was to fix this eheh.
i think it would make more sense to just change

if(!params->PathBuffer && !params->PathBufferSize)

to

if(!params->PathBuffer)

as it was talked about in irc, it doesn't make much sense to check for
pathbuffersize when pathbuffer is null...
the max_path removal was a mistake from following msdn documentation :\

2009/3/22 James Hawkins <truiken at gmail.com>

> ---
>  dlls/setupapi/dialog.c |   12 +++++++-----
>  1 files changed, 7 insertions(+), 5 deletions(-)
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20090322/d97da074/attachment.htm>


More information about the wine-devel mailing list