bugGNUstep - Bugs: bug #30218, NSURL's initWithCoder doesn't...

Group
 
 

bug #30218: NSURL's initWithCoder doesn't account for rel being nil in the archive

Submitter:  Thomas Davie <beelsebob>
Submitted:  Tue 22 Jun 2010 04:07:17 PM UTC
   
 
Category:  Base/Foundation Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 05 Mar 2011 03:15:32 PM UTC, comment #3: 

No update after several months ... assume fixed.

Richard Frith-Macdonald <CaS>
Group Member
Fri 25 Jun 2010 04:01:39 PM UTC, comment #2: 

I implemented keyed archiving for NSURL.
I expect that fixes the issue.

Richard Frith-Macdonald <CaS>
Group Member
Wed 23 Jun 2010 07:11:28 AM UTC, comment #1: 

I'm not sure about the problem diagnosis here ...
'Some archives produced by apple contain NSURL with no relative string'

The GNUstep binary archive format is not compatible with Apple's one (and NSURL doesn't support keyed archiving), so I can't see how this can appear.

Perhaps the actual problem is a lack of keyed archiving support and we need to reverse engineer that support from some apple archives?

Richard Frith-Macdonald <CaS>
Group Member
Tue 22 Jun 2010 04:07:17 PM UTC, original submission:  

Some archives produced by apple contain NSURL with no relative string, this causes a crash, because initWithString:relativeToURL: may not accept nil as it's first argument

I've attached a patch to fix it.

My employer Hunted Cow Studios Ltd. asked me to note that I created this patch on their time :).

Thomas Davie <beelsebob>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #20791:  url.diff added by beelsebob (578B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by CaS (Posted a comment)
  • -email is unavailable- added by beelsebob (Submitted the item)
  •  

    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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-03-05 CaS Open/ClosedIn Test Closed
    2010-06-25 CaS StatusNone Fixed
        Open/ClosedOpen In Test
    2010-06-22 beelsebob Attached File- Added url.diff, #20791

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code