New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

memory: add missing curl_printf header #2999

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@danielgustafsson
Member

danielgustafsson commented Sep 15, 2018

ftp_send_command() was using vsnprintf() without including the libcurl *rintf() replacement header. Fix by including curl_printf.h.

@bagder

Thanks!

Show resolved Hide resolved lib/security.c Outdated
memory: add missing curl_printf header
ftp_send_command() was using vsnprintf() without including the libcurl
*rintf() replacement header. Fix by including curl_printf.h and also
add curl_memory.h while at it since memdebug.h depends on it.
@bagder

This comment has been minimized.

Member

bagder commented Sep 16, 2018

Hm, test 2100 seems a bit flaky :-( (restarted the failed travis build since that failure was hardly related to the commit)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment