From 5f36942fd3196259ad5ad691f5ecbe4ce555ab7c Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Sat, 10 Oct 2026 00:29:02 +0300 Subject: [PATCH] Fix check on test function's arguments The function won't cope with nkinds == 0, it leads to division by zero. It's just a test function though, so this is harmless and there's no need to backport. Author: Yuriy Grigoryev Reviewed-by: Sho Ito Discussion: https://www.postgresql.org/message-id/1905e9d709554733b418d06a4806436f@localhost.localdomain --- src/test/modules/test_resowner/test_resowner_many.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/test/modules/test_resowner/test_resowner_many.c b/src/test/modules/test_resowner/test_resowner_many.c index a3b49bbbcf2..e3873bc57a1 100644 --- a/src/test/modules/test_resowner/test_resowner_many.c +++ b/src/test/modules/test_resowner/test_resowner_many.c @@ -213,14 +213,14 @@ test_resowner_many(PG_FUNCTION_ARGS) ManyTestResourceKind *after_kinds; /* Sanity check the arguments */ - if (nkinds < 0) - elog(ERROR, "nkinds must be >= 0"); + if (nkinds <= 0) + elog(ERROR, "nkinds must be > 0"); if (nremember_bl < 0) elog(ERROR, "nremember_bl must be >= 0"); if (nforget_bl < 0 || nforget_bl > nremember_bl) elog(ERROR, "nforget_bl must between 0 and 'nremember_bl'"); if (nremember_al < 0) - elog(ERROR, "nremember_al must be greater than zero"); + elog(ERROR, "nremember_al must be >= 0"); if (nforget_al < 0 || nforget_al > nremember_al) elog(ERROR, "nforget_al must between 0 and 'nremember_al'");