[PATCH] dpnet: Initialize size variable. (Coverity)
Sven Baars
sven.wine at gmail.com
Sun Jun 16 05:01:51 CDT 2019
On 16-06-19 08:49, Alistair Leslie-Hughes wrote:
> Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
> ---
> dlls/dpnet/peer.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/dlls/dpnet/peer.c b/dlls/dpnet/peer.c
> index f935b2a313..3df07749c5 100644
> --- a/dlls/dpnet/peer.c
> +++ b/dlls/dpnet/peer.c
> @@ -157,6 +157,7 @@ HRESULT enum_services_providers(const GUID * const service, DPN_SERVICE_PROVIDER
> next_key = RegEnumKeyW( key, index, provider, MAX_PATH);
> while(next_key == ERROR_SUCCESS)
> {
> + size = 0;
> res = RegGetValueW(key, provider, friendly, RRF_RT_REG_SZ, NULL, NULL, &size);
> if(res == ERROR_SUCCESS)
> {
Wouldn't it be more correct to fix RegGetValueW to do
cbData = (pcbData && pvData) ? *pcbData : 0;
instead of
cbData = pcbData ? *pcbData : 0;
instead? RegQueryValueExW does something similar.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20190616/97d7ccfe/attachment.html>
More information about the wine-devel
mailing list