diff options
| author | Carlos Martín Nieto <cmn@dwim.me> | 2013-03-08 02:11:34 +0100 |
|---|---|---|
| committer | Carlos Martín Nieto <cmn@dwim.me> | 2013-03-09 15:45:18 +0100 |
| commit | 48bde2f1b62d24f3982382d520bfac887537641d (patch) | |
| tree | da7dde9bafe3725298d7de04e6b52f351cc6c054 /tests-clar/online/fetchhead.c | |
| parent | 92ebbe99c9b557aee3382f51ddf6dd6637ee2fe4 (diff) | |
| download | libgit2-48bde2f1b62d24f3982382d520bfac887537641d.tar.gz | |
config: don't allow passing NULL as a value to set
Passing NULL is non-sensical. The error message leaves to be desired,
though, as it leaks internal implementation details. Catch it at the
`git_config_set_string` level and set an appropriate error message.
Diffstat (limited to 'tests-clar/online/fetchhead.c')
| -rw-r--r-- | tests-clar/online/fetchhead.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests-clar/online/fetchhead.c b/tests-clar/online/fetchhead.c index 84a2177ea..a8a5bb918 100644 --- a/tests-clar/online/fetchhead.c +++ b/tests-clar/online/fetchhead.c @@ -79,8 +79,8 @@ void test_online_fetchhead__no_merges(void) fetchhead_test_clone(); cl_git_pass(git_repository_config(&config, g_repo)); - cl_git_pass(git_config_set_string(config, "branch.master.remote", NULL)); - cl_git_pass(git_config_set_string(config, "branch.master.merge", NULL)); + cl_git_pass(git_config_delete_entry(config, "branch.master.remote")); + cl_git_pass(git_config_delete_entry(config, "branch.master.merge")); git_config_free(config); fetchhead_test_fetch(NULL, FETCH_HEAD_NO_MERGE_DATA); |