Re: d3dx9_36/tests: Fix sizeof to be more portable (coverity)

[Henri Verbeet]

On 24 October 2012 02:05, André Hentschel  wrote:
>      face_remap = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY,
> -
>                           tc[i].num_faces*sizeof(face_remap));
> -
>                           tc[i].num_faces*sizeof(DWORD*));

I'm don't know what Coverity actually says about this, but this looks
just as wrong to me. It's probably supposed to be
"sizeof(*face_remap)".

d3dx9_36/tests: Fix sizeof to be more portable (coverity)

[André Hentschel]

Am 24.10.2012 02:32, schrieb Henri Verbeet:
> On 24 October 2012 02:05, André Hentschel  wrote:
> >      face_remap = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY,
> > -
> >                           tc[i].num_faces*sizeof(face_remap));
> > -
> >                           tc[i].num_faces*sizeof(DWORD*));
>
> I'm don't know what Coverity actually says about this, but this looks
> just as wrong to me. It's probably supposed to be
> "sizeof(*face_remap)".

That's actually what Coverity suggested, but i thought i've seen more sizeofs of this kind.