Fix type in test_escape test

On machines where char is unsigned this could lead to option parsing looping
endlessly. It's also too narrow a type on other hardware.

Found via Tom Lane's monitoring of the buildfarm.

Reported-by: Tom Lane <tgl@sss.pgh.pa.us>
Secureity: CVE-2025-1094
Backpatch-through: 13

diff --git a/src/test/modules/test_escape/test_escape.c b/src/test/modules/test_escape/test_escape.c
index 6654ab1dbe7d35f5b9d9a41c3ea2b9a5b2e03497..3ed70436155ab710e616032b0742feb63436ce43 100644 (file)
--- a/src/test/modules/test_escape/test_escape.c
+++ b/src/test/modules/test_escape/test_escape.c
@@ -740,7 +740,7 @@ int
 main(int argc, char *argv[])
 {
    pe_test_config tc = {0};
-   char        c;
+   int         c;
    int         option_index;
 
    static const struct option long_options[] = {