gnustep/libs-base
0
0
Fork 0
mirror of https://github.com/gnustep/libs-base.git synced 2025-04-23 09:04:13 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix to prevent crash (raise exception instead) when deserializing a corrupt file

Browse source 
git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@21379 72102866-910b-0410-8b05-ffd578937521
This commit is contained in:
Richard Frith-Macdonald 2005-06-30 07:44:45 +00:00
parent 750130f0e6
commit 9b8bcaa3e6
2 changed files with 19 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
ChangeLog
View file

@ -1,3 +1,10 @@
2005-06-30 Richard Frith-Macdonald <rfm@gnu.org>
* Source/NSSerializer.m: deserializeFromInfo() check that cross
reference indices are valid and raise an exception if not ... should
fix bug reported by Roland Schwingel, where trying to deserialize a
corrupt file could cause a crash.
2005-06-28 Adam Fedor <fedor@gnu.org>
* Source/NSInvocation.m (-getReturnValue:, -setReturnValue:): Only

14
Source/NSSerializer.m
View file

@ -520,7 +520,9 @@ static void
endDeserializerInfo(_NSDeserializerInfo* info)
{
if (info->didUnique)
GSIArrayEmpty(&info->array);
{
GSIArrayEmpty(&info->array);
}
}
static id
@ -536,7 +538,15 @@ deserializeFromInfo(_NSDeserializerInfo* info)
case ST_XREF:
{
size = (*info->deiImp)(info->data, deiSel, info->cursor);
return RETAIN(GSIArrayItemAtIndex(&info->array, size).obj);
if (size < GSIArrayCount(&info->array))
{
return RETAIN(GSIArrayItemAtIndex(&info->array, size).obj);
}
else
{
[NSException raise: NSInvalidArgumentException
format: @"Bad cross reference in property list"];
}
}
case ST_CSTRING: