Skip to content

Commit

Permalink
upstream: another case where a utimes() failure could make scp send
Browse files Browse the repository at this point in the history
a desynchronising error; reminded by Aymeric Vincent ok deraadt markus

OpenBSD-Commit-ID: 2ea611d34d8ff6d703a7a8bf858aa5dbfbfa7381
  • Loading branch information
djmdjm authored and daztucker committed May 7, 2020
1 parent 59d5315 commit 955854c
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions scp.c
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/* $OpenBSD: scp.c,v 1.209 2020/05/01 06:31:42 djm Exp $ */
/* $OpenBSD: scp.c,v 1.210 2020/05/06 20:57:38 djm Exp $ */
/*
* scp - secure remote copy. This is basically patched BSD rcp which
* uses ssh to do the data transfer (instead of using rcmd).
Expand Down Expand Up @@ -1427,9 +1427,7 @@ sink(int argc, char **argv, const char *src)
sink(1, vect, src);
if (setimes) {
setimes = 0;
if (utimes(vect[0], tv) == -1)
run_err("%s: set times: %s",
vect[0], strerror(errno));
(void) utimes(vect[0], tv);
}
if (mod_flag)
(void) chmod(vect[0], mode);
Expand Down

0 comments on commit 955854c

Please sign in to comment.