[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix SIGSEGV in svn_path_is_uri_safe

From: Branko Čibej <brane_at_xbc.nu>
Date: 2003-08-06 06:44:41 CEST

kfogel@collab.net wrote:

>Branko Čibej <brane@xbc.nu> writes:
>
>
>>I seem to remember us having a policy that it's the caller's
>>responsibility to pass valid parameters to a function -- in this case,
>>the caller shouldn't call path_is_uri_safe at all if the parameter is
>>NULL. Has this policy changed?
>>
>>
>
>Which parameter do you mean?
>
>The bug here was not that the parameter to svn_path_is_uri_safe() was
>NULL, but that we were failing to check the result of a strchr() call
>for NULL. In other words, just an implementation braino, no deeper
>implications as far as I can tell...
>
>
Yikes. Shows what happens if you "review" stuff in the middle of the
night. Sorry.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Aug 6 06:47:36 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.