From 6984aa3a45a9a31b35e41e03e2dd32b8c2c97152 Mon Sep 17 00:00:00 2001 From: Stefan Eissing Date: Thu, 8 Feb 2024 17:52:29 +0100 Subject: [PATCH] ftp: tracing improvements - trace socketindex for connection filters when not the first - trace socket fd in tcp - trace pollset adjusts in vtls Closes #12902 --- lib/cf-socket.c | 11 +++++++---- lib/curl_trc.c | 6 +++++- lib/vtls/vtls.c | 4 ++++ 3 files changed, 16 insertions(+), 5 deletions(-) diff --git a/lib/cf-socket.c b/lib/cf-socket.c index 742902f1b9..33ca7e7818 100644 --- a/lib/cf-socket.c +++ b/lib/cf-socket.c @@ -880,8 +880,9 @@ static ssize_t nw_in_read(void *reader_ctx, nread = -1; } } - CURL_TRC_CF(rctx->data, rctx->cf, "nw_in_read(len=%zu) -> %d, err=%d", - len, (int)nread, *err); + CURL_TRC_CF(rctx->data, rctx->cf, "nw_in_read(len=%zu, fd=%" + CURL_FORMAT_SOCKET_T ") -> %d, err=%d", + len, ctx->sock, (int)nread, *err); return nread; } @@ -1248,11 +1249,13 @@ static void cf_socket_adjust_pollset(struct Curl_cfilter *cf, if(ctx->sock != CURL_SOCKET_BAD) { if(!cf->connected) { Curl_pollset_set_out_only(data, ps, ctx->sock); - CURL_TRC_CF(data, cf, "adjust_pollset(!connected) -> %d socks", ps->num); + CURL_TRC_CF(data, cf, "adjust_pollset, !connected, POLLOUT fd=%" + CURL_FORMAT_SOCKET_T, ctx->sock); } else if(!ctx->active) { Curl_pollset_add_in(data, ps, ctx->sock); - CURL_TRC_CF(data, cf, "adjust_pollset(!active) -> %d socks", ps->num); + CURL_TRC_CF(data, cf, "adjust_pollset, !active, POLLIN fd=%" + CURL_FORMAT_SOCKET_T, ctx->sock); } } } diff --git a/lib/curl_trc.c b/lib/curl_trc.c index b8dccc4193..2f7fb65fdd 100644 --- a/lib/curl_trc.c +++ b/lib/curl_trc.c @@ -134,7 +134,11 @@ void Curl_trc_cf_infof(struct Curl_easy *data, struct Curl_cfilter *cf, va_list ap; int len; char buffer[MAXINFO + 2]; - len = msnprintf(buffer, MAXINFO, "[%s] ", cf->cft->name); + if(cf->sockindex) + len = msnprintf(buffer, MAXINFO, "[%s-%d] ", + cf->cft->name, cf->sockindex); + else + len = msnprintf(buffer, MAXINFO, "[%s] ", cf->cft->name); va_start(ap, fmt); len += mvsnprintf(buffer + len, MAXINFO - len, fmt, ap); va_end(ap); diff --git a/lib/vtls/vtls.c b/lib/vtls/vtls.c index f654a9749c..51596d80b8 100644 --- a/lib/vtls/vtls.c +++ b/lib/vtls/vtls.c @@ -774,9 +774,13 @@ void Curl_ssl_adjust_pollset(struct Curl_cfilter *cf, struct Curl_easy *data, if(sock != CURL_SOCKET_BAD) { if(connssl->connecting_state == ssl_connect_2_writing) { Curl_pollset_set_out_only(data, ps, sock); + CURL_TRC_CF(data, cf, "adjust_pollset, POLLOUT fd=%" + CURL_FORMAT_SOCKET_T, sock); } else { Curl_pollset_set_in_only(data, ps, sock); + CURL_TRC_CF(data, cf, "adjust_pollset, POLLIN fd=%" + CURL_FORMAT_SOCKET_T, sock); } } }