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:59 -0500 |
| commit | 4a6825c762112f085d6ee03bbce8e7abe2bf98e5 (patch) | |
| tree | b8c535d5ca75021fc3a6372fd740785b33f2e62a | |
| parent | 17b789469106e156f559fcdab0c720db3c38151a (diff) | |
| download | postgresql-4a6825c762112f085d6ee03bbce8e7abe2bf98e5.tar.gz postgresql-4a6825c762112f085d6ee03bbce8e7abe2bf98e5.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[] = { |