Horton

Thread

Commits

  1. test_bloomfilter: Fix error message.

  1. BUG: test_bloomfilter error message reports wrong variable and wrong format specifier

    Jianghua Yang <yjhjstz@gmail.com> — 2026-03-24T12:31:55Z

    Hi,

  I found a small bug in
src/test/modules/test_bloomfilter/test_bloomfilter.c, line 128.

  The current code:

  int64   nelements = PG_GETARG_INT64(1);
  int     tests = PG_GETARG_INT32(3);

  if (tests <= 0)
      elog(ERROR, "invalid number of tests: %d", tests);

  if (nelements < 0)
      elog(ERROR, "invalid number of elements: %d", tests);

  The second elog has two issues:

  1. It checks nelements but prints tests. For example, with
  nelements = -1 and tests = 1, the error message would be
  "invalid number of elements: 1" — which is misleading when debugging.
  2. nelements is int64, so the format specifier should be
  INT64_FORMAT rather than %d. Using %d for int64 is
  undefined behavior on 32-bit platforms.

  The fix:

  if (nelements < 0)
      elog(ERROR, "invalid number of elements: " INT64_FORMAT, nelements);

  A patch is attached.

  Regards,
  Jianghua Yang

  2. Re: BUG: test_bloomfilter error message reports wrong variable and wrong format specifier

    Nathan Bossart <nathandbossart@gmail.com> — 2026-03-24T14:34:23Z

    On Tue, Mar 24, 2026 at 05:31:55AM -0700, Jianghua Yang wrote:
>   A patch is attached.

Committed, thanks.

-- 
nathan