diff options
| author | Andres Freund <andres@anarazel.de> | 2025-02-10 12:09:23 -0500 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2025-02-10 12:12:53 -0500 |
| commit | 05abb0f8303a78921f7113bee1d72586142df99e (patch) | |
| tree | c3f92eceb993dc48e7ed0e7d337144820d164b0f | |
| parent | dd3c1eb38e9add293f8be59b6aec7574e8584bdb (diff) | |
| download | postgresql-05abb0f8303a78921f7113bee1d72586142df99e.tar.gz postgresql-05abb0f8303a78921f7113bee1d72586142df99e.zip | |
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>
Security: CVE-2025-1094
Backpatch-through: 13
| -rw-r--r-- | src/test/modules/test_escape/test_escape.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/modules/test_escape/test_escape.c b/src/test/modules/test_escape/test_escape.c index 6654ab1dbe7..3ed70436155 100644 --- 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[] = { |