X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=lib-lib%2Ffile.c;h=67b1d741c9c985364d3997ba6c0ca8c3177a0977;hp=985015f0da26fb904e350b4f4fef0017d8481d8b;hb=3692b834c97c9933088d7082464fec5ae903920f;hpb=ac813896ca32d850febc2d95065ac4fa040f11f9 diff --git a/lib-lib/file.c b/lib-lib/file.c index 985015f..67b1d74 100644 --- a/lib-lib/file.c +++ b/lib-lib/file.c @@ -37,7 +37,6 @@ #include "mem.h" #include "str.h" #include "file.h" -#include "debug.h" #ifndef O_NOFOLLOW # define O_NOFOLLOW 0 @@ -75,7 +74,6 @@ int safe_open(const char *path, int flags) if (lstat (path, &osb) < 0 || fstat(fd, &nsb) < 0 || compare_stat(&osb, &nsb) == -1) { - debug_print(1, ("%s is a symlink!\n", path)); close(fd); return -1; } @@ -254,6 +252,43 @@ int safe_fclose(FILE **f) return r; } +/* If rfc1524_expand_command() is used on a recv'd message, then + * the filename doesn't exist yet, but if its used while sending a message, + * then we need to rename the existing file. + * + * This function returns 0 on successful move, 1 on old file doesn't exist, + * 2 on new file already exists, and 3 on other failure. + */ + +/* note on access(2) use: No dangling symlink problems here due to + * safe_fopen(). + */ +int mutt_rename_file(char *oldfile, char *newfile) +{ + FILE *ofp, *nfp; + + if (access(oldfile, F_OK) != 0) + return 1; + if (access(newfile, F_OK) == 0) + return 2; + + ofp = fopen(oldfile, "r"); + if (!ofp) + return 3; + + nfp = safe_fopen(newfile, "w"); + if (!nfp) { + fclose (ofp); + return 3; + } + + mutt_copy_stream(ofp, nfp); + fclose(nfp); + fclose(ofp); + mutt_unlink(oldfile); + return 0; +} + /* Read a line from ``fp'' into the dynamically allocated ``s'', * increasing ``s'' if necessary. The ending "\n" or "\r\n" is removed. * If a line ends with "\", this char and the linefeed is removed, @@ -328,7 +363,6 @@ int mutt_copy_bytes(FILE *in, FILE *out, ssize_t size) if ((chunk = fread(buf, 1, chunk, in)) < 1) break; if (fwrite(buf, 1, chunk, out) != chunk) { - debug_print(1, ("fwrite() returned short byte count\n")); return -1; } size -= chunk;