From 19134112bf942c0492f2095b668ea31c1a498f1a Mon Sep 17 00:00:00 2001 From: Lars Hjemli Date: Sun, 24 Feb 2008 15:27:33 +0100 Subject: Add regression-test for description-less repos The segfault fixed in commit eacde43d7184452e1fdc90b982b531f1f5239923 was triggered when the html-functions manipulated string literals. One callpatch which could trigger the bug is in ui-repolist.c when repo descriptions are passed to html_ntxt(): if a repo is lacking a description, the literal string "[no description]" is used. This patch changes test/setup.sh such that the first repo has no description, and adds tests for both "[no description]" and "the bar repo" (description of the other repo) to tests/t0101-index.sh, which should be enought to catch regressions in these functions. Noticed-by: Hiroki Hattori Signed-off-by: Lars Hjemli --- tests/setup.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'tests/setup.sh') diff --git a/tests/setup.sh b/tests/setup.sh index 51d5a75..9bb1f12 100755 --- a/tests/setup.sh +++ b/tests/setup.sh @@ -54,7 +54,9 @@ summary-tags=5 repo.url=foo repo.path=$PWD/trash/repos/foo/.git -repo.desc=the foo repo +# Do not specify a description for this repo, as it then will be assigned +# the constant value "[no description]" (which actually used to cause a +# segfault). repo.url=bar repo.path=$PWD/trash/repos/bar/.git -- cgit v1.2.3-59-g8ed1b