Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit52ef33f

Browse files
committed
Ensure libpq reports a suitable error message on unexpected socket EOF.
The EOF-detection logic in pqReadData was a bit confused about who shouldset up the error message in case the kernel gives us read-ready-but-no-datarather than ECONNRESET or some other explicit error condition. Since thewhole point of this situation is that the lower-level functions don't knowthere's anything wrong, pqReadData itself must set up the message. Butkeep the assumption that if an errno was reported, a message was set up atlower levels.Per bug #11712 from Marko Tiikkaja. It's been like this for a very longtime, so back-patch to all supported branches.
1 parentd7624e5 commit52ef33f

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

‎src/interfaces/libpq/fe-misc.c‎

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -763,12 +763,8 @@ pqReadData(PGconn *conn)
763763
/* ready for read */
764764
break;
765765
default:
766-
printfPQExpBuffer(&conn->errorMessage,
767-
libpq_gettext(
768-
"server closed the connection unexpectedly\n"
769-
"\tThis probably means the server terminated abnormally\n"
770-
"\tbefore or while processing the request.\n"));
771-
gotodefinitelyFailed;
766+
/* we override pqReadReady's message with something more useful */
767+
gotodefinitelyEOF;
772768
}
773769

774770
/*
@@ -807,9 +803,16 @@ pqReadData(PGconn *conn)
807803

808804
/*
809805
* OK, we are getting a zero read even though select() says ready. This
810-
* means the connection has been closed. Cope. Note that errorMessage
811-
* has been set already.
806+
* means the connection has been closed. Cope.
812807
*/
808+
definitelyEOF:
809+
printfPQExpBuffer(&conn->errorMessage,
810+
libpq_gettext(
811+
"server closed the connection unexpectedly\n"
812+
"\tThis probably means the server terminated abnormally\n"
813+
"\tbefore or while processing the request.\n"));
814+
815+
/* Come here if lower-level code already set a suitable errorMessage */
813816
definitelyFailed:
814817
pqDropConnection(conn);
815818
conn->status=CONNECTION_BAD;/* No more connection to backend */

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp