Hey,

On 2014-10-31 10:22, akha...@svn.reactos.org wrote:
> @@ -90,6 +91,7 @@
>                       //Invalid parameter detected
>                       if (AllocAndLoadString(&lpIllegalMsg, 
> GetModuleHandle(NULL), IDS_ILLEGAL_PARAM))
>                       _putts(lpIllegalMsg);
> +                     LocalFree(lpIllegalMsg);
>                       return FALSE;
>               }
>       }
> 

This needs braces. Both puts and LocalFree should only be executed if
AllocAndLoadString succeeds (but return FALSE in either case).

Thanks!

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to