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:56 -0500 |
| commit | 41343f84052eb53a900ea464a8ce16a548cab901 (patch) | |
| tree | 37f9846bd9883d8bdabe5a8e304d6c95364b3cc3 | |
| parent | 16ce519533bd03db726953db2364877a8944da4c (diff) | |
| download | postgresql-41343f84052eb53a900ea464a8ce16a548cab901.tar.gz postgresql-41343f84052eb53a900ea464a8ce16a548cab901.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[] = { |