| author | Denys Vlasenko <vda.linux@googlemail.com> | 2016-08-13 21:23:48 (GMT) |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2016-08-13 21:23:48 (GMT) |
| commit | 98c50f93fe494be5547188813c367299db80dbc5 (patch) | |
| tree | 061399d7f505fb806919c46385c38d74c1c80e88 | |
| parent | 150dc7a2b483b8338a3e185c478b4b23ee884e71 (diff) | |
| download | busybox-98c50f93fe494be5547188813c367299db80dbc5.zip busybox-98c50f93fe494be5547188813c367299db80dbc5.tar.gz busybox-98c50f93fe494be5547188813c367299db80dbc5.tar.bz2 | |
cp: fix -i for POSIX mode. Closes 9106
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rw-r--r-- | libbb/copy_file.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/libbb/copy_file.c b/libbb/copy_file.c index 7801b58..23c0f83 100644 --- a/libbb/copy_file.c +++ b/libbb/copy_file.c @@ -296,11 +296,16 @@ int FAST_FUNC copy_file(const char *source, const char *dest, int flags) if (!S_ISREG(source_stat.st_mode)) new_mode = 0666; - // POSIX way is a security problem versus (sym)link attacks - if (!ENABLE_FEATURE_NON_POSIX_CP) { - dst_fd = open(dest, O_WRONLY|O_CREAT|O_TRUNC, new_mode); - } else { /* safe way: */ + if (ENABLE_FEATURE_NON_POSIX_CP || (flags & FILEUTILS_INTERACTIVE)) { + /* + * O_CREAT|O_EXCL: require that file did not exist before creation + */ dst_fd = open(dest, O_WRONLY|O_CREAT|O_EXCL, new_mode); + } else { /* POSIX, and not "cp -i" */ + /* + * O_CREAT|O_TRUNC: create, or truncate (security problem versus (sym)link attacks) + */ + dst_fd = open(dest, O_WRONLY|O_CREAT|O_TRUNC, new_mode); } if (dst_fd == -1) { ovr = ask_and_unlink(dest, flags); |