]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
lib: msnprintf tidy-ups
authorDaniel Stenberg <daniel@haxx.se>
Mon, 28 Oct 2024 15:55:13 +0000 (16:55 +0100)
committerDaniel Stenberg <daniel@haxx.se>
Mon, 28 Oct 2024 21:56:11 +0000 (22:56 +0100)
doh: avoid an msnprintf()

openssl: skip a superfluous return code check

Closes #15434

lib/doh.c
lib/vtls/openssl.c

index 1873d413b70d3720c0dc24be4222d1f352c1a584..8769372e0be6f6769c8267e29f3ac4257ff065ea 100644 (file)
--- a/lib/doh.c
+++ b/lib/doh.c
@@ -847,11 +847,9 @@ static void doh_show(struct Curl_easy *data,
     }
     else if(a->type == DNS_TYPE_AAAA) {
       int j;
-      char buffer[128];
-      char *ptr;
-      size_t len;
-      len = msnprintf(buffer, 128, "[DoH] AAAA: ");
-      ptr = &buffer[len];
+      char buffer[128] = "[DoH] AAAA: ";
+      size_t len = strlen(buffer);
+      char *ptr = &buffer[len];
       len = sizeof(buffer) - len;
       for(j = 0; j < 16; j += 2) {
         size_t l;
index e380c2e08e4ff995f01e85bd3682bc41fbff29d5..67c7fd0f790086d2f66b02c0a4270ac8827b85bd 100644 (file)
@@ -2683,9 +2683,7 @@ static void ossl_trace(int direction, int ssl_ver, int content_type,
                         "%s (%s), %s, %s (%d):\n",
                         verstr, direction ? "OUT" : "IN",
                         tls_rt_name, msg_name, msg_type);
-    if(0 <= txt_len && (unsigned)txt_len < sizeof(ssl_buf)) {
-      Curl_debug(data, CURLINFO_TEXT, ssl_buf, (size_t)txt_len);
-    }
+    Curl_debug(data, CURLINFO_TEXT, ssl_buf, (size_t)txt_len);
   }
 
   Curl_debug(data, (direction == 1) ? CURLINFO_SSL_DATA_OUT :