Author: breser
Date: Fri Aug 9 08:23:12 2013
New Revision: 1512196
URL: http://svn.apache.org/r1512196
Log:
Add an assert for the working_url in checkout_file().
This member of the struct is allowed to be NULL for non-checked out dirs.
But checkout_file() assumes that it is always true (which it should be).
If this assumption fails at least we'll get an error this way.
* subversion/libsvn_ra_serf/commit.c
(checkout_file): Add SVN_ERR_ASSERT on parent_dir->working_url.
Modified:
subversion/trunk/subversion/libsvn_ra_serf/commit.c
Modified: subversion/trunk/subversion/libsvn_ra_serf/commit.c
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/commit.c?rev=1512196&r1=1512195&r2=1512196&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/commit.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/commit.c Fri Aug 9 08:23:12 2013
@@ -552,6 +552,7 @@ checkout_file(file_context_t *file,
if (parent_dir->added)
{
/* Implicitly checkout this file now. */
+ SVN_ERR_ASSERT(parent_dir->working_url);
file->working_url = svn_path_url_add_component2(
parent_dir->working_url,
svn_relpath_skip_ancestor(
