Re: [ros-dev] [ros-diffs] [akhaldi] 65133: [LOGOFF] * Plug some leaks. CIDs 716284 and 716285. CORE-7975

[Thomas Faber]

Hey,

On 2014-10-31 10:22, akhaldi(a)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!