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

Commit21a86b0

Browse files
committed
postgres_fdw: Fix unexpected reporting of empty message.
pgfdw_report_error() in postgres_fdw gets a message from PGresult orPGconn to report an error received from a remote server. Previouslyif it could get a message from neither of them, it reported emptymessage unexpectedly. The cause of this issue was that pgfdw_report_error()didn't handle properly the case where no message could be obtainedand its local variable message_primary was set to '\0'.This commit improves pgfdw_report_error() so that it reports the message"could not obtain ..." when it gets no message and message_primaryis set to '\0'. This is the same behavior as when message_primary is NULL.dblink_res_error() in dblink has the same issue, so this commit alsoimproves it in the same way.Back-patch to all supported branches.Author: Fujii MasaoReviewed-by: Bharath RupireddyDiscussion:https://postgr.es/m/477c16c8-7ea4-20fc-38d5-ed3a77ed616c@oss.nttdata.com
1 parent3e644dc commit21a86b0

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

‎contrib/dblink/dblink.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2757,7 +2757,8 @@ dblink_res_error(PGconn *conn, const char *conname, PGresult *res,
27572757

27582758
ereport(level,
27592759
(errcode(sqlstate),
2760-
message_primary ?errmsg_internal("%s",message_primary) :
2760+
(message_primary!=NULL&&message_primary[0]!='\0') ?
2761+
errmsg_internal("%s",message_primary) :
27612762
errmsg("could not obtain message string for remote error"),
27622763
message_detail ?errdetail_internal("%s",message_detail) :0,
27632764
message_hint ?errhint("%s",message_hint) :0,

‎contrib/postgres_fdw/connection.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -626,7 +626,8 @@ pgfdw_report_error(int elevel, PGresult *res, PGconn *conn,
626626

627627
ereport(elevel,
628628
(errcode(sqlstate),
629-
message_primary ?errmsg_internal("%s",message_primary) :
629+
(message_primary!=NULL&&message_primary[0]!='\0') ?
630+
errmsg_internal("%s",message_primary) :
630631
errmsg("could not obtain message string for remote error"),
631632
message_detail ?errdetail_internal("%s",message_detail) :0,
632633
message_hint ?errhint("%s",message_hint) :0,

0 commit comments

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