From a972a8b72bdb56bfb9dca1d7c3c9291d5832e081 Mon Sep 17 00:00:00 2001 From: lum Date: Wed, 24 Feb 2021 13:58:46 +0000 Subject: [PATCH] snprintf() does indeed return an int. From an email Joachim Nilsson sent to tech@ many moons ago. Sorry for the delay. --- usr.bin/mg/dired.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/usr.bin/mg/dired.c b/usr.bin/mg/dired.c index 69c806f4ebc..90d9a638373 100644 --- a/usr.bin/mg/dired.c +++ b/usr.bin/mg/dired.c @@ -1,4 +1,4 @@ -/* $OpenBSD: dired.c,v 1.93 2019/07/11 18:20:18 lum Exp $ */ +/* $OpenBSD: dired.c,v 1.94 2021/02/24 13:58:46 lum Exp $ */ /* This file is in the public domain. */ @@ -479,9 +479,9 @@ d_copy(int f, int n) topath = adjustname(toname, TRUE); if (stat(topath, &statbuf) == 0) { if (S_ISDIR(statbuf.st_mode)) { - off = snprintf(toname, sizeof(toname), "%s/%s", + ret = snprintf(toname, sizeof(toname), "%s/%s", topath, sname); - if (off < 0 || off >= sizeof(toname) - 1) { + if (ret < 0 || ret >= sizeof(toname) - 1) { dobeep(); ewprintf("Directory name too long"); return (FALSE); -- 2.20.1