diff options
| author | Andres Freund | 2010-02-14 10:24:45 +0000 |
|---|---|---|
| committer | Andres Freund | 2010-02-14 23:02:07 +0000 |
| commit | 0dc602ca41fee1ca8bd85056add8bcb8f44bf510 (patch) | |
| tree | ff4096fb6410f0824725a5c821e12b15168d30a4 | |
| parent | 528b1f060e6863c48e372c838d474c8339ac160d (diff) | |
Dont try to save the errno in RecoveryConflictInterrupt to avoid confusion.feature/hs-cleanup
In the current state the errno saving in there is confusing as there
are several returns ignoring to set it. I think its currently harmless
as there should be no changes to errno at those places - beside that
the only caller (procsignal_sigusr1_handler) already saves it.
| -rw-r--r-- | src/backend/tcop/postgres.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 8c0c8b9c88..3505fc430c 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -2753,11 +2753,9 @@ SigHupHandler(SIGNAL_ARGS) void RecoveryConflictInterrupt(ProcSignalReason reason) { - int save_errno = errno; - /* - * Don't joggle the elbow of proc_exit - */ + * Don't joggle the elbow of proc_exit + */ if (!proc_exit_inprogress) { RecoveryConflictReason = reason; @@ -2856,8 +2854,6 @@ RecoveryConflictInterrupt(ProcSignalReason reason) ProcessInterrupts(); } } - - errno = save_errno; } /* |