postgres.git / summary / log / commit / refs

commit    07abcd9ab712b843568e2f9a00006dd25797dc02
Author:   Michael Paquier <michael@paquier.xyz>
Date:     Mon Aug 01 07:39:16 2022 +0000
First released in: 16.0, 12.12, 13.8, 14.5, 15.0

    Fix error reporting after ioctl() call with pg_upgrade --clone
    
    errno was not reported correctly after attempting to clone a file,
    leading to incorrect error reports.  While scanning through the code, I
    have not noticed any similar mistakes.
    
    Error introduced in 3a769d8.
    
    Author: Justin Pryzby
    Discussion: https://postgr.es/m/20220731134135.GY15006@telsasoft.com
    Backpatch-through: 12


src/bin/pg_upgrade/file.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/bin/pg_upgrade/file.c b/src/bin/pg_upgrade/file.c index b84868c7510..e16c2e009eb 100644 --- a/src/bin/pg_upgrade/file.c +++ b/src/bin/pg_upgrade/file.c @@ -57,9 +57,11 @@ cloneFile(const char *src, const char *dst, if (ioctl(dest_fd, FICLONE, src_fd) < 0) { + int save_errno = errno; + unlink(dst); pg_fatal("error while cloning relation \"%s.%s\" (\"%s\" to \"%s\"): %s\n", - schemaName, relName, src, dst, strerror(errno)); + schemaName, relName, src, dst, strerror(save_errno)); } close(src_fd); [parent: feae5c0cba80]