Re: msxml3: Add ISAXContentHandler_endPrefix event (1/7)

[Alexandre Julliard]

Piotr Caban  writes:
> +static HRESULT namespacePush(saxlocator *locator, int ns)
> +{
> - if(locator->nsStackLast+1>locator->nsStackSize)
> - {
> -
>    locator->nsStackSize += STACKALLOCATIONSIZE;
> -
>    locator->nsStack = HeapReAlloc(GetProcessHeap(), 0,
> -
>            locator->nsStack, locator->nsStackSize);
> -
>    if(!locator->nsStack) return E_FAIL;

You should preserve the existing stack on failure. Also the growth
should be by a multiple of the current size instead of a constant.