Hi...
Richard Frith-Macdonald <[EMAIL PROTECTED]> wrote on 30.06.2005
10:34:27:
> > <[EMAIL PROTECTED]> wrote:>
> > Unfortunately it does not work and the reason is clear. With my
corrupt file
> > info->didUnique is set to 0 in initDeserializerInfo(), so the
GSIArray is
> > also not initialized and calling GSIArrayCount(&info->array) accesses
> > uninitialized data. Access to the GSIArray should only be done after
> > checking the didUnique flag.
>
> Thanks ... I missed that the array could be uninitialised. Should be
fixed
> now to check both that the array is initialised and that the index
into it
> is valid (and raise a different exception message in each case).
With one last small tweak it works now. I added a
break;
after the last else branch you added to terminate the ST_XREF case,
elsewise it would rush directly into the ST_CSTRING case which is
bad.
Thanks,
Roland
_______________________________________________
Bug-gnustep mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-gnustep
