diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-09-26 12:35:57 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-09-26 12:35:57 -0400 |
| commit | 758ce9b7794845f95473c569155d29fcf0e2751b (patch) | |
| tree | 9110f776f5f665b4cfff55223c7e0f2da47ffaaf /src/interfaces/libpq/fe-auth.c | |
| parent | 26e9d4d4ef16b5e2be96319f89ea6ba7f63a4d73 (diff) | |
Incorporate strerror_r() into src/port/snprintf.c, too.
This provides the features that used to exist in useful_strerror()
for users of strerror_r(), too. Also, standardize on the GNU convention
that strerror_r returns a char pointer that may not be NULL.
I notice that libpq's win32.c contains a variant version of strerror_r
that probably ought to be folded into strerror.c. But lacking a
Windows environment, I should leave that to somebody else.
Discussion: https://postgr.es/m/2975.1526862605@sss.pgh.pa.us
Diffstat (limited to 'src/interfaces/libpq/fe-auth.c')
| -rw-r--r-- | src/interfaces/libpq/fe-auth.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/interfaces/libpq/fe-auth.c b/src/interfaces/libpq/fe-auth.c index 540aba98b37..92641fe5e9a 100644 --- a/src/interfaces/libpq/fe-auth.c +++ b/src/interfaces/libpq/fe-auth.c @@ -756,11 +756,11 @@ pg_local_sendauth(PGconn *conn) if (sendmsg(conn->sock, &msg, 0) == -1) { - char sebuf[256]; + char sebuf[PG_STRERROR_R_BUFLEN]; printfPQExpBuffer(&conn->errorMessage, "pg_local_sendauth: sendmsg: %s\n", - pqStrerror(errno, sebuf, sizeof(sebuf))); + strerror_r(errno, sebuf, sizeof(sebuf))); return STATUS_ERROR; } return STATUS_OK; @@ -1098,7 +1098,7 @@ pg_fe_getauthname(PQExpBuffer errorMessage) printfPQExpBuffer(errorMessage, libpq_gettext("could not look up local user ID %d: %s\n"), (int) user_id, - pqStrerror(pwerr, pwdbuf, sizeof(pwdbuf))); + strerror_r(pwerr, pwdbuf, sizeof(pwdbuf))); else printfPQExpBuffer(errorMessage, libpq_gettext("local user with ID %d does not exist\n"), |