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

Re: svn_client_commit with empty targets array => coredump

From: Branko Čibej <brane_at_xbc.nu>
Date: 2002-08-25 12:37:48 CEST

Marcus Comstedt wrote:

>I just noticed that calling svn_client_commit with an empty array for
>the targets argument causes a core dump. The reason is that
>svn_path_condense_targets() will set both base_dir and rel_targets to
>NULL if the array is empty, and svn_client_commit() doesn't like this
>at all. Which of the functions should be considered at fault here?
>Is there any particular reason why svn_path_condense_targets() on an
>empty array doesn't just set base_dir and rel_targets to the empty
>string and an empty array respectively? Setting them to NULL seems to
>compilcate the edge cases for no obvious benefit.

Do svn_client_commit (and other APIs that accept dir/array pairs) handle
the empty/empty case? If they do, then I think your plan is fine.

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 Sun Aug 25 12:38:30 2002

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.