diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:09:52 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:18:03 +0100 |
| commit | 604bd3671121b51f977de146ed95484c2297fb3e (patch) | |
| tree | 829045453c054befb9d795c33765501e2f4522f3 /src/backend/libpq/be-fsstubs.c | |
| parent | 73025140885c889410b9bfc4a30a3866396fc5db (diff) | |
| download | postgresql-604bd3671121b51f977de146ed95484c2297fb3e.tar.gz | |
PG_FINALLY
This gives an alternative way of catching exceptions, for the common
case where the cleanup code is the same in the error and non-error
cases. So instead of
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_CATCH();
{
cleanup();
PG_RE_THROW();
}
PG_END_TRY();
cleanup();
one can write
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_FINALLY();
{
cleanup();
}
PG_END_TRY();
Discussion: https://www.postgresql.org/message-id/flat/95a822c3-728b-af0e-d7e5-71890507ae0c%402ndquadrant.com
Diffstat (limited to 'src/backend/libpq/be-fsstubs.c')
| -rw-r--r-- | src/backend/libpq/be-fsstubs.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/backend/libpq/be-fsstubs.c b/src/backend/libpq/be-fsstubs.c index b0ece7ec25..1ee96c0b9f 100644 --- a/src/backend/libpq/be-fsstubs.c +++ b/src/backend/libpq/be-fsstubs.c @@ -503,13 +503,11 @@ be_lo_export(PG_FUNCTION_ARGS) fd = OpenTransientFilePerm(fnamebuf, O_CREAT | O_WRONLY | O_TRUNC | PG_BINARY, S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH); } - PG_CATCH(); + PG_FINALLY(); { umask(oumask); - PG_RE_THROW(); } PG_END_TRY(); - umask(oumask); if (fd < 0) ereport(ERROR, (errcode_for_file_access(), |