diff options
| author | Jeff Davis <jdavis@postgresql.org> | 2024-09-24 12:01:45 -0700 |
|---|---|---|
| committer | Jeff Davis <jdavis@postgresql.org> | 2024-09-24 12:01:45 -0700 |
| commit | ceeaaed87aa39425f1f2c2409f927c76efb8de89 (patch) | |
| tree | 4ddc189ec3606636cdb9e1c995cbfe6e31568de8 /src/backend/executor/execParallel.c | |
| parent | 59f0eea7b001e3f5d38fd84a9ee5482a23be56e2 (diff) | |
| download | postgresql-ceeaaed87aa39425f1f2c2409f927c76efb8de89.tar.gz postgresql-ceeaaed87aa39425f1f2c2409f927c76efb8de89.zip | |
Tighten up make_libc_collator() and make_icu_collator().
Ensure that error paths within these functions do not leak a collator,
and return the result rather than using an out parameter. (Error paths
in the caller may still result in a leaked collator, which will be
addressed separately.)
In make_libc_collator(), if the first newlocale() succeeds and the
second one fails, close the first locale_t object.
The function make_icu_collator() doesn't have any external callers, so
change it to be static.
Discussion: https://postgr.es/m/54d20e812bd6c3e44c10eddcd757ec494ebf1803.camel@j-davis.com
Diffstat (limited to 'src/backend/executor/execParallel.c')
0 files changed, 0 insertions, 0 deletions