[PATCH] Fix repos->capability assumption in repos hook

Hi,

It seems svn_repos_remember_client_capabilities is only called in the
ra server side, in other words, if you use the repos api, you can get
a repos object without client_capabilities set which is optional as
repos.h indicates.

However, the svn_repos__hooks_start_commit function assumes capability
is not null, which causes segfault if you have a repository with
enabled start-commit hooks and is being committed with the repos API.
Attached is a patch for this.

I am also wondering why the code has the caller (the only one is
svn_repos_fs_begin_txn_for_commit2) passing the capabilities in rather
than pulling it from repos? the latter seems more encapsulated to
me...

[[[

* subversion/libsvn_repos/hook.c
  (svn_repos__hooks_start_commit): Don't assume capabilities is non-null.

]]]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org