On Fri, Jan 14, 2011 at 12:32 PM, Danny Trebbien <dtrebbien_at_gmail.com> wrote:
> Attached are a patch and log message to extend
> svn_subst_translate_string2() with a new parameter, REPAIR. This
> allows the caller to customize whether the translation repairs line
I noticed a slight problem with the extra asserts in
subversion/tests/libsvn_subr/subst_translate-test.c that this patch
adds. When REPAIR is FALSE and VALUE has inconsistent line endings,
then an SVN_ERR_IO_INCONSISTENT_EOL error is returned. If, however, a
future change causes svn_subst_translate_string2() to return
SVN_NO_ERROR, then the assert that the error was
SVN_ERR_IO_INCONSISTENT_EOL would dereference a NULL pointer.
Attached are the patch and log message for version 1.1 of this patch.
The new patch corrects this flaw. The new log message corrects a
Received on 2011-01-15 02:58:23 CET