From 6a5bf66a42110c0415715acfed80bb1771a9844f Mon Sep 17 00:00:00 2001 From: Joachim Wiberg Date: Sun, 23 Feb 2025 16:01:58 +0100 Subject: [PATCH] Fix #956: skip chown() on running-config, not applicable Signed-off-by: Joachim Wiberg --- src/bin/copy.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/bin/copy.c b/src/bin/copy.c index c16368d6..1649c28c 100644 --- a/src/bin/copy.c +++ b/src/bin/copy.c @@ -83,11 +83,14 @@ static void set_owner(const char *fn, const char *user) { struct passwd *pw; + if (!fn) + return; /* not an error, e.g., running-config is not a file */ + pw = getpwnam(user); if (pw && !chmod(fn, 0660) && !chown(fn, pw->pw_uid, pw->pw_gid)) return; - fprintf(stderr, ERRMSG "setting owner %s on %s: %s\n", fn, user, strerror(errno)); + fprintf(stderr, ERRMSG "setting owner %s on %s: %s\n", user, fn, strerror(errno)); } static const char *infix_ds(const char *text, struct infix_ds **ds)