On Sun, Aug 01, 2010 at 10:39:04AM +0200, Bert Huijben wrote:
> > + if (APR_STATUS_IS_ENOENT(err->apr_err))
> > + {
> > + /* 1.6.0 to 1.6.11 did not copy the configuration file
> > during
> > + * hotcopy. So if we're hotcopying a repository which
> > has been
> > + * created as a hotcopy itself, it's possible that
> > fsfs.conf
> > + * does not exist. Ask the user to re-create it.
> > + *
> > + * ### It would be nice to make this a non-fatal error,
> > + * ### but this function does not get an svn_fs_t object
> > + * ### so we have no way of just printing a warning via
> > + * ### the fs->warning() callback. */
> > +
> > + const char *msg;
> > + const char *src_abspath;
> > + const char *dst_abspath;
> > + const char *config_relpath;
> > +
>
> You leak the error in err here.
No, because we quick_wrap it later:
> > + return svn_error_return(svn_error_quick_wrap(err, msg));
^^^^^^^^^^^^^^^^^^^^^^^^
> > + }
> > + else
> > + return svn_error_return(err);
> > + }
> > + }
> > +
