Reported by Dan Kegel

Reverts b474649e. Some games use
pointers to the guid rather than copying the guid even after the
enumeration function returns. Allocating on the stack breaks those
games.

Perhaps the code should also just double the buffer. I can send another patch
for that if needed.

It doesn't make sense to only do this when the buffer is resized.

Found by Valgrind.

user32: Don't try to free a handle with a value of 1, which is the dde handle value for asynchronous operations.

For non-recorded stateblocks a light always exists in the device stateblock as
well, since that's where it's copied from. However, when SetLightEnable() is
called for a light that didn't a exist yet while recording a stateblock, the
light is only created on the recorded stateblock.

ole2: Convert the icon for OleMetafilePictFromIconAndLabel to 32-bit explicitly instead of casting the handle.