From a1767a1f5d744804958b2ef0516cbd5d33292c0c Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sat, 14 Jun 2008 04:28:41 +0000 Subject: [PATCH] dos2unix: do not destroy symlinks and mode of the file being converted. --- coreutils/dos2unix.c | 33 ++++++++++++++++++--------------- libbb/xreadlink.c | 1 - 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/coreutils/dos2unix.c b/coreutils/dos2unix.c index 2db7e11a1..311dc1380 100644 --- a/coreutils/dos2unix.c +++ b/coreutils/dos2unix.c @@ -24,24 +24,27 @@ static void convert(char *fn, int conv_type) { FILE *in, *out; int i; - char *name_buf = name_buf; /* for compiler */ + char *temp_fn = temp_fn; /* for compiler */ + char *resolved_fn = resolved_fn; in = stdin; out = stdout; if (fn != NULL) { - in = xfopen(fn, "r"); - /* - The file is then created with mode read/write and - permissions 0666 for glibc 2.0.6 and earlier or - 0600 for glibc 2.0.7 and later. - */ - name_buf = xasprintf("%sXXXXXX", fn); - i = mkstemp(name_buf); + struct stat st; + + resolved_fn = xmalloc_follow_symlinks(fn); + if (resolved_fn == NULL) + bb_simple_perror_msg_and_die(fn); + in = xfopen(resolved_fn, "r"); + fstat(fileno(in), &st); + + temp_fn = xasprintf("%sXXXXXX", resolved_fn); + i = mkstemp(temp_fn); if (i == -1 - || fchmod(i, 0600) == -1 + || fchmod(i, st.st_mode) == -1 || !(out = fdopen(i, "w+")) ) { - bb_perror_nomsg_and_die(); + bb_simple_perror_msg_and_die(temp_fn); } } @@ -56,12 +59,12 @@ static void convert(char *fn, int conv_type) if (fn != NULL) { if (fclose(in) < 0 || fclose(out) < 0) { - unlink(name_buf); + unlink(temp_fn); bb_perror_nomsg_and_die(); } -// TODO: destroys symlinks. See how passwd handles this - xrename(name_buf, fn); - free(name_buf); + xrename(temp_fn, resolved_fn); + free(temp_fn); + free(resolved_fn); } } diff --git a/libbb/xreadlink.c b/libbb/xreadlink.c index 706a3d9ff..0b961b6f9 100644 --- a/libbb/xreadlink.c +++ b/libbb/xreadlink.c @@ -54,7 +54,6 @@ char *xmalloc_follow_symlinks(const char *path) goto jump_in; while (1) { - linkpath = xmalloc_readlink(buf); if (!linkpath) { /* not a symlink, or doesn't exist */ -- 2.25.1