#include "actions.h"
static int
-fwrite_cb (void *fp_ptr, const void *buf, int len)
+write_cb (void *fd_ptr, const void *buf, size_t len)
{
- FILE *fp = *(FILE **)fp_ptr;
- return fwrite (buf, len, 1, fp) == 1 ? 0 : -1;
+ int fd = *(int *)fd_ptr;
+ return xwrite (fd, buf, len);
}
/* Has one FileIn parameter. */
}
free (cmd);
- r = receive_file (fwrite_cb, &fp);
+ /* The semantics of fwrite are too undefined, so write to the
+ * file descriptor directly instead.
+ */
+ int fd = fileno (fp);
+
+ r = receive_file (write_cb, &fd);
if (r == -1) { /* write error */
- err = errno;
cancel_receive ();
- errno = err;
- reply_with_perror ("write: %s", dir);
+ reply_with_error ("write error on directory: %s", dir);
pclose (fp);
return -1;
}
}
if (pclose (fp) != 0) {
- err = errno;
- cancel_receive ();
- errno = err;
- reply_with_perror ("pclose: %s", dir);
+ if (r == -1) /* if r == 0, file transfer ended already */
+ cancel_receive ();
+ reply_with_error ("tar subcommand failed on directory: %s", dir);
return -1;
}
}
free (cmd);
- r = receive_file (fwrite_cb, &fp);
+ int fd = fileno (fp);
+
+ r = receive_file (write_cb, &fd);
if (r == -1) { /* write error */
- err = errno;
cancel_receive ();
- errno = err;
- reply_with_perror ("write: %s", dir);
+ reply_with_error ("write error on directory: %s", dir);
pclose (fp);
return -1;
}
}
if (pclose (fp) != 0) {
- err = errno;
- cancel_receive ();
- errno = err;
- reply_with_perror ("pclose: %s", dir);
+ if (r == -1) /* if r == 0, file transfer ended already */
+ cancel_receive ();
+ reply_with_error ("tar subcommand failed on directory: %s", dir);
return -1;
}