On Wed, Jul 1, 2009 at 3:18 PM, David Blanton<aired...@tularosa.net> wrote: > Here is code that crashes. the parms coming in are checked for validity > before ReplaceChar is called > > void ReplaceChar(char *s, char oldChar, char newChar) > { > if(newChar == oldChar) > return; > char *n = strchr(s, oldChar); > while(n) > { > *n = newChar; > n = strchr(n+1, oldChar); > } > }
Are you sure "oldChar" cannot be '\0'? If "oldChar" is '\0' then your n+1 will put you past the end "s". Of course replacing the null terminator with anything other then a null terminator will break the c-stringness of the character array. Anyway can you post a full backtrace of a crashed thread? -Shawn _______________________________________________ Cocoa-dev mailing list (Cocoa-dev@lists.apple.com) Please do not post admin requests or moderator comments to the list. Contact the moderators at cocoa-dev-admins(at)lists.apple.com Help/Unsubscribe/Update your Subscription: http://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com This email sent to arch...@mail-archive.com