diff mbox series

[v7,04/16] reftable: check reftable_stack_auto_compact() return value

Message ID b0bffb6e29d2d060c7698cd3e84918f14ad8a3e3.1643051624.git.gitgitgadget@gmail.com (mailing list archive)
State Accepted
Commit f7445865f260474082c8276dc2175ec23109c424
Headers show
Series Reftable coverity fixes | expand

Commit Message

Han-Wen Nienhuys Jan. 24, 2022, 7:13 p.m. UTC 
From: Han-Wen Nienhuys <hanwen@google.com>

If compaction consistently errors out, we would accumulate reftables,
and the follow-on EXPECT would start failing, but catching errors
early helps debugging.

Spotted by Coverity.

Signed-off-by: Han-Wen Nienhuys <hanwen@google.com>
---
 reftable/stack_test.c | 1 +
 1 file changed, 1 insertion(+)
diff mbox series

Patch

diff --git a/reftable/stack_test.c b/reftable/stack_test.c
index f4c743db80c..e84f50d27ff 100644
--- a/reftable/stack_test.c
+++ b/reftable/stack_test.c
@@ -839,6 +839,7 @@  static void test_reftable_stack_auto_compaction(void)
 		EXPECT_ERR(err);
 
 		err = reftable_stack_auto_compact(st);
+		EXPECT_ERR(err);
 		EXPECT(i < 3 || st->merged->stack_len < 2 * fastlog2(i));
 	}