cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [Patch] vtls/polarssl.c

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Sat, 22 Mar 2014 16:58:19 +0100 (CET)

On Sat, 22 Mar 2014, Gisle Vanem wrote:

> The debug messages printed inside PolarSSL always seems to end with a
> newline. So 'infof()' should not add one. Besides the trace 'line' should be
> 'const'. Patch:

Thanks, I pushed it with a little edit:

> - infof(data, "%s\n", line);
> + infof(data, line);

I prefer to keep that:

  infof(data, "%s", line);

... to avoid the risk if "line" contains a %s in itself at some point.

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2014-03-22