bugGNUstep - Bugs: bug #17097, Possible NSKeyedArchiver Bug

Group
 
 

bug #17097: Possible NSKeyedArchiver Bug

Submitter:  Michael Johnston <mjohnston>
Submitted:  Thu 13 Jul 2006 06:03:08 PM UTC
   
 
Category:  Base/Foundation Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 25 Jul 2006 04:28:09 PM UTC, comment #4: 

Someone overwrote the fix ... I've reinstated it.

Richard Frith-Macdonald <CaS>
Group Member
Tue 25 Jul 2006 02:37:51 PM UTC, comment #3: 

I just checked out the latest source and it seems as though the change hasn't been commited.

Michael Johnston <mjohnston>
Fri 14 Jul 2006 10:55:57 AM UTC, comment #2: 

Thanks ... fixed in SVN using GSIArrayEmpty() as suggested.

Richard Frith-Macdonald <CaS>
Group Member
Fri 14 Jul 2006 08:47:09 AM UTC, comment #1: 

Sorry meant NSKeyedUnarchiver.

Michael Johnston <mjohnston>
Thu 13 Jul 2006 06:03:08 PM UTC, original submission:  

I've noticed that objects returned by NSKeyedArchiver using decodeObjectForKey: are implicitly retained by the decoder. I thought that objects returned by decodeObjectForKey: should be autoreleased. Is this correct behaviour?

Looking at NSKeyedArchiver source the object is retained when
it is added to the _objMap array in method _decodeObject of NSKeyedArchivers Private category (Theres a comment "Retained in array). However on dealloc the array is cleared using GSIArrayClear which as far as I can see clears the memory but doesnt release the objects in the array.

Maybe GSIArrayEmpty is what should be used here?

Michael Johnston <mjohnston>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 2 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-07-14 CaS StatusNone Fixed
    Open/ClosedOpen Closed

Back to the top

Powered by Savane 3.13-d3ae.
Corresponding source code