diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-26 15:30:10 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-12-26 15:30:10 -0500 |
| commit | e9fcfed3fb6b15e6b33bea9bb8504b14560857a8 (patch) | |
| tree | 446e7367eb042030c7540ea46d6a1f6b8a3973ec /src/common/psprintf.c | |
| parent | f89ae34ab8b4d9e9ce8af6bd889238b0ccff17cb (diff) | |
| download | postgresql-e9fcfed3fb6b15e6b33bea9bb8504b14560857a8.tar.gz postgresql-e9fcfed3fb6b15e6b33bea9bb8504b14560857a8.zip | |
Fix portability failure introduced in commits d2b0b60e7 et al.
I made a frontend fprintf() format use %m, forgetting that that's only
safe in HEAD not the back branches; prior to 96bf88d52 and d6c55de1f,
it would work on glibc platforms but not elsewhere. Revert to using
%s ... strerror(errno) as the code did before.
We could have left HEAD as-is, but for code consistency across branches,
I chose to apply this patch there too.
Per Coverity and a few buildfarm members.
Diffstat (limited to 'src/common/psprintf.c')
| -rw-r--r-- | src/common/psprintf.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/common/psprintf.c b/src/common/psprintf.c index 411713bac84..cdbc04fe39a 100644 --- a/src/common/psprintf.c +++ b/src/common/psprintf.c @@ -115,7 +115,8 @@ pvsnprintf(char *buf, size_t len, const char *fmt, va_list args) #ifndef FRONTEND elog(ERROR, "vsnprintf failed: %m with format string \"%s\"", fmt); #else - fprintf(stderr, "vsnprintf failed: %m with format string \"%s\"\n", fmt); + fprintf(stderr, "vsnprintf failed: %s with format string \"%s\"\n", + strerror(errno), fmt); exit(EXIT_FAILURE); #endif } |