In trunk (and in the 1.6 backport), the close_edit() function in the subject does a simple 'return err' when svn_repos_fs_commit_txn() fails.
That err may be SVN_NO_ERROR. Shouldn't close_edit() return a non-NULL error even if svn_repos_fs_commit_txn() failed but returned SVN_NO_ERROR ?