[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnustep-cvs] gnustep/dev-libs/gsweb/GSWeb.framework GSWDynam...

From: David Ayers
Subject: Re: [Gnustep-cvs] gnustep/dev-libs/gsweb/GSWeb.framework GSWDynam...
Date: Sat, 01 Oct 2005 09:48:24 +0200
User-agent: Mozilla Thunderbird 1.0.2 (X11/20050817)

David Wetzel schrieb:
> CVSROOT:      /cvsroot/gnustep
> Module name:  gnustep
> Branch:       
> Changes by:   David Wetzel <address@hidden>   05/09/30 15:30:00
> Modified files:
>       dev-libs/gsweb/GSWeb.framework: GSWDynamicURLString.m 
> Log message:
>       added - (NSData*) dataUsingEncoding: (NSStringEncoding)encoding 
> allowLossyConversion: (BOOL)flag
> CVSWeb URLs:

Even though this is the right thing to do for GSWDynamicURLString, I
believe this is may be covering up a bug in the generic implementation
of dataUsingEncoding:allowLossyConversion: of NSString.  .... and as I'm
looking at the code to explain what the issue is, the bug jumps at me:

The NSData was created with the freed (larger) buffer and not the one
returned by the realloc for UTF8 conversions.

Could you locally update -base and revert your fix just to test that
this was the issue?  (Well I'm about 99.9998% sure that this is the
issue but there may still be another issue hiding.)  But we do want to
keep your implementation of GSWDynamicURLString.


PS: A test for the test suite would also be great so if someone has the
time to implement a minimal custom string class...

reply via email to

[Prev in Thread] Current Thread [Next in Thread]