aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/sendf.c20
1 files changed, 11 insertions, 9 deletions
diff --git a/lib/sendf.c b/lib/sendf.c
index 11914d5e8..89f9af28d 100644
--- a/lib/sendf.c
+++ b/lib/sendf.c
@@ -60,8 +60,8 @@
#ifdef KRB4
#include "security.h"
-#include <string.h>
#endif
+#include <string.h>
/* The last #include file should be: */
#ifdef MALLOCDEBUG
#include "memdebug.h"
@@ -123,37 +123,39 @@ size_t sendf(int fd, struct UrlData *data, char *fmt, ...)
/*
* ftpsendf() sends the formated string as a ftp command to a ftp server
+ *
+ * NOTE: we build the command in a fixed-length buffer, which sets length
+ * restrictions on the command!
+ *
*/
size_t ftpsendf(int fd, struct connectdata *conn, char *fmt, ...)
{
size_t bytes_written;
- char *s;
+ char s[256];
+
va_list ap;
va_start(ap, fmt);
- s = mvaprintf(fmt, ap);
+ vsnprintf(s, 250, fmt, ap);
va_end(ap);
- if(!s)
- return 0; /* failure */
+
if(conn->data->bits.verbose)
fprintf(conn->data->err, "> %s\n", s);
+ strcat(s, "\r\n"); /* append a trailing CRLF */
+
#ifdef KRB4
if(conn->sec_complete && conn->data->cmdchannel) {
bytes_written = sec_fprintf(conn, conn->data->cmdchannel, s);
- bytes_written += fprintf(conn->data->cmdchannel, "\r\n");
fflush(conn->data->cmdchannel);
}
else
#endif /* KRB4 */
{
bytes_written = swrite(fd, s, strlen(s));
- bytes_written += swrite(fd, "\r\n", 2);
}
- free(s); /* free the output string */
return(bytes_written);
}
-
/* ssend() sends plain (binary) data to the server */
size_t ssend(int fd, struct connectdata *conn, void *mem, size_t len)
{