[PATCH] fusion: use correct size for ZeroMemory (GCC7)
Marcus Meissner
meissner at suse.de
Fri Jun 2 04:38:33 CDT 2017
On Fri, Jun 02, 2017 at 04:58:50PM +0800, Dmitry Timoshkov wrote:
> Marcus Meissner <meissner at suse.de> wrote:
>
> > diff --git a/dlls/fusion/tests/asmname.c b/dlls/fusion/tests/asmname.c
> > index 5fb14a4829..21cd4874ed 100644
> > --- a/dlls/fusion/tests/asmname.c
> > +++ b/dlls/fusion/tests/asmname.c
> > @@ -358,7 +358,7 @@ static void test_assembly_name_props_line(IAssemblyName *name,
> > to_widechar(expect, vals[i].val);
> >
> > size = MAX_PATH;
> > - ZeroMemory(str, MAX_PATH);
> > + ZeroMemory(str, sizeof(str));
> > hr = IAssemblyName_GetProperty(name, i, str, &size);
> > to_multibyte(val, str);
>
> Same problem with IAssemblyName_GetProperty() and 'size' argumenet as
> for the previous patch.
I am not sure documentation and reality agree on this.
If I pass in size in bytes and then later on change it back to character count
the testsuite starts to fail with 2* factors.
Our implementation and upstream(?) seems to return character count on some of those
properties.
Ciao, Marcus
More information about the wine-devel
mailing list