btrfs-progs: send: check for output file existence before creating
authorDavid Sterba <dsterba@suse.com>
Mon, 7 Nov 2016 15:51:20 +0000 (16:51 +0100)
committerDavid Sterba <dsterba@suse.com>
Fri, 11 Nov 2016 15:23:29 +0000 (16:23 +0100)
In some cases the root might not be able to create the output file (and
streaming to stdout is not an option). Make the output file creation two
step and let it work if the file is already created.

Signed-off-by: David Sterba <dsterba@suse.com>
cmds-send.c

index fbb2f80..7aa4b36 100644 (file)
@@ -571,7 +571,20 @@ int cmd_send(int argc, char **argv)
                usage(cmd_send_usage);
 
        if (outname[0]) {
-               send.dump_fd = creat(outname, 0600);
+               int tmpfd;
+
+               /*
+                * Try to use an existing file first. Even if send runs as
+                * root, it might not have permissions to create file (eg. on a
+                * NFS) but it should still be able to use a pre-created file.
+                */
+               tmpfd = open(outname, O_WRONLY | O_TRUNC);
+               if (tmpfd < 0) {
+                       if (errno == ENOENT)
+                               tmpfd = open(outname,
+                                       O_CREAT | O_WRONLY | O_TRUNC, 0600);
+               }
+               send.dump_fd = tmpfd;
                if (send.dump_fd == -1) {
                        ret = -errno;
                        error("cannot create '%s': %s", outname, strerror(-ret));