From e77e98b5cf0a8e52d879bf06c68574e971f88253 Mon Sep 17 00:00:00 2001 From: dtucker Date: Fri, 3 Sep 2021 05:12:25 +0000 Subject: [PATCH] Remove redundant attrib_clear in upload_dir_internal. The subsequent call to stat_to_attrib clears the struct as its first step anyway. From pmeinhardt via github PR#220, ok djm@ --- usr.bin/ssh/sftp-client.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/usr.bin/ssh/sftp-client.c b/usr.bin/ssh/sftp-client.c index 8667475d509..ffd6d92c85f 100644 --- a/usr.bin/ssh/sftp-client.c +++ b/usr.bin/ssh/sftp-client.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sftp-client.c,v 1.154 2021/08/09 23:47:44 djm Exp $ */ +/* $OpenBSD: sftp-client.c,v 1.155 2021/09/03 05:12:25 dtucker Exp $ */ /* * Copyright (c) 2001-2004 Damien Miller * @@ -2014,7 +2014,6 @@ upload_dir_internal(struct sftp_conn *conn, const char *src, const char *dst, if (print_flag && print_flag != SFTP_PROGRESS_ONLY) mprintf("Entering %s\n", src); - attrib_clear(&a); stat_to_attrib(&sb, &a); a.flags &= ~SSH2_FILEXFER_ATTR_SIZE; a.flags &= ~SSH2_FILEXFER_ATTR_UIDGID; -- 2.20.1