diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2016-10-08 13:39:29 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2016-10-08 15:13:46 +0200 |
| commit | 8238ba9c5f10414a88f502bf3f5d5a42d632984c (patch) | |
| tree | eeeb5d93d5435b83f678a54bdffd52d0c287c846 /lib/security.c | |
| parent | 9885c9508ec757f7f658dab11658e4a3e643a420 (diff) | |
| download | curl-8238ba9c5f10414a88f502bf3f5d5a42d632984c.tar.gz | |
ftp: fix Curl_ftpsendf()
... it no longer takes printf() arguments since it was only really taken
advantage by one user and it was not written and used in a safe
way. Thus the 'f' is removed from the function name and the proto is
changed.
Although the current code wouldn't end up in badness, it was a risk that
future changes could end up springf()ing too large data or passing in a
format string inadvertently.
Diffstat (limited to 'lib/security.c')
| -rw-r--r-- | lib/security.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/security.c b/lib/security.c index a0bcaeaee..a268d4a62 100644 --- a/lib/security.c +++ b/lib/security.c @@ -122,7 +122,7 @@ static int ftp_send_command(struct connectdata *conn, const char *message, ...) vsnprintf(print_buffer, sizeof(print_buffer), message, args); va_end(args); - if(Curl_ftpsendf(conn, print_buffer)) { + if(Curl_ftpsend(conn, print_buffer)) { ftp_code = -1; } else { |