From 10beb36b1cd1479d14b245a922e1ab49d9a8b1f9 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sat, 18 Feb 2006 22:27:01 +0000 Subject: [PATCH] =?utf8?q?Ulf=20H=E4rnhammar=20fixed=20a=20format=20string?= =?utf8?q?=20(printf=20style)=20problem=20in=20the=20Negotiate=20code.=20I?= =?utf8?q?t=20should=20however=20not=20be=20the=20cause=20of=20any=20troub?= =?utf8?q?les.=20He=20also=20fixed=20a=20few=20similar=20problems=20in=20t?= =?utf8?q?he=20HTTP=20test=20server=20code.?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- CHANGES | 5 +++++ RELEASE-NOTES | 3 ++- lib/http_negotiate.c | 2 +- tests/server/sws.c | 6 +++--- 4 files changed, 11 insertions(+), 5 deletions(-) diff --git a/CHANGES b/CHANGES index cde84d3..084b6c5 100644 --- a/CHANGES +++ b/CHANGES @@ -6,6 +6,11 @@ Changelog +Daniel (18 February 2006) +- Ulf Härnhammar fixed a format string (printf style) problem in the Negotiate + code. It should however not be the cause of any troubles. He also fixed a + few similar problems in the HTTP test server code. + Daniel (17 February 2006) - Shmulik Regev provided a fix for the DNS cache when using short life times, as previously it could be holding on to old cached entries longer than diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 4b73a2f..521502b 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -21,6 +21,7 @@ This release includes the following changes: This release includes the following bugfixes: + o minor format string mistake in the GSS/Negotiate code o cached DNS entries could remain in the cache too long o improved GnuTLS check in configure o re-used FTP connections when the second request didn't do a transfer @@ -63,6 +64,6 @@ advice from friends like these: Dov Murik, Jean Jacques Drouin, Andres Garcia, Yang Tse, Gisle Vanem, Dan Fandrich, Alexander Lazic, Michael Jahn, Andrew Benham, Bryan Henderson, David Shaw, Jon Turner, Duane Cathey, Michal Marek, Philippe Vaucher, Kent - Boortz, Karl Moerder, Shmulik Regev + Boortz, Karl Moerder, Shmulik Regev, Ulf Härnhammar Thanks! (and sorry if I forgot to mention someone) diff --git a/lib/http_negotiate.c b/lib/http_negotiate.c index f859a8b..f407d50 100644 --- a/lib/http_negotiate.c +++ b/lib/http_negotiate.c @@ -111,7 +111,7 @@ log_gss_error(struct connectdata *conn, OM_uint32 error_status, char *prefix) gss_release_buffer(&min_stat, &status_string); } while (!GSS_ERROR(maj_stat) && msg_ctx != 0); - infof(conn->data, buf); + infof(conn->data, "%s", buf); } int Curl_input_negotiate(struct connectdata *conn, char *header) diff --git a/tests/server/sws.c b/tests/server/sws.c index b1904b4..4b6735c 100644 --- a/tests/server/sws.c +++ b/tests/server/sws.c @@ -219,7 +219,7 @@ int ProcessRequest(struct httprequest *req) else sprintf(logbuf, "Got a *HUGE* request HTTP/%d.%d", prot_major, prot_minor); - logmsg(logbuf); + logmsg("%s", logbuf); if(!strncmp("/verifiedserver", ptr, 15)) { logmsg("Are-we-friendly question received"); @@ -251,7 +251,7 @@ int ProcessRequest(struct httprequest *req) sprintf(logbuf, "Requested test number %ld part %ld", req->testno, req->partno); - logmsg(logbuf); + logmsg("%s", logbuf); filename = test2file(req->testno); @@ -294,7 +294,7 @@ int ProcessRequest(struct httprequest *req) doc, &prot_major, &prot_minor) == 3) { sprintf(logbuf, "Receiced a CONNECT %s HTTP/%d.%d request", doc, prot_major, prot_minor); - logmsg(logbuf); + logmsg("%s", logbuf); if(prot_major*10+prot_minor == 10) req->open = FALSE; /* HTTP 1.0 closes connection by default */ -- 2.7.4