Change a malloc() to a HeapAlloc()

[Peter Davies]

> We use real names here.  This patch probably will not be committed until it
> is resubmitted.
Ops! I've mis-configured my mail client

> The patch is also wrong.
Why? What difference does malloc() over HeapAlloc() make? (this is arguably a
case for not sending the patch in the first place)

There is nothing that bugs me more than not being told why I'm wrong.

Peter