Commit 82d7a114eb56c64da15dc278c90385dd3f83c28c

Patrick Steinhardt 2019-11-05T11:18:14

Merge pull request #5293 from csware/config_snapshot-snapshot Fix crash if snapshotting a config_snapshot

diff --git a/src/config_snapshot.c b/src/config_snapshot.c
index 764abe2..e89a138 100644
--- a/src/config_snapshot.c
+++ b/src/config_snapshot.c
@@ -192,6 +192,7 @@ int git_config_backend_snapshot(git_config_backend **out, git_config_backend *so
 	backend->parent.get = config_get_readonly;
 	backend->parent.set = config_set_readonly;
 	backend->parent.set_multivar = config_set_multivar_readonly;
+	backend->parent.snapshot = git_config_backend_snapshot;
 	backend->parent.del = config_delete_readonly;
 	backend->parent.del_multivar = config_delete_multivar_readonly;
 	backend->parent.iterator = config_iterator_new_readonly;
diff --git a/tests/config/snapshot.c b/tests/config/snapshot.c
index 61562d2..3b90cfe 100644
--- a/tests/config/snapshot.c
+++ b/tests/config/snapshot.c
@@ -100,3 +100,23 @@ void test_config_snapshot__includes(void)
 	cl_git_pass(p_unlink("including"));
 	cl_git_pass(p_unlink("included"));
 }
+
+void test_config_snapshot__snapshot(void)
+{
+	git_config *snapshot_snapshot;
+	int i;
+
+	cl_git_mkfile("configfile", "[section]\nkey = 1\n");
+
+	cl_git_pass(git_config_open_ondisk(&cfg, "configfile"));
+	cl_git_pass(git_config_snapshot(&snapshot, cfg));
+
+	cl_git_pass(git_config_snapshot(&snapshot_snapshot, snapshot));
+
+	cl_git_pass(git_config_get_int32(&i, snapshot_snapshot, "section.key"));
+	cl_assert_equal_i(i, 1);
+
+	git_config_free(snapshot_snapshot);
+
+	cl_git_pass(p_unlink("configfile"));
+}