This is an automated email from the ASF dual-hosted git repository.

dmeden pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 9dce7846c3 Coverity: Fix for CID-1587277 (#11950)
9dce7846c3 is described below

commit 9dce7846c3e7e79763083051ff9754b9b35ea08e
Author: Damian Meden <[email protected]>
AuthorDate: Tue Jan 14 10:45:04 2025 +0100

    Coverity: Fix for CID-1587277 (#11950)
    
    JA4, zero initialization for TLS version member variable.
---
 plugins/experimental/ja4_fingerprint/ja4.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/experimental/ja4_fingerprint/ja4.h 
b/plugins/experimental/ja4_fingerprint/ja4.h
index 743b867250..31b151dc0c 100644
--- a/plugins/experimental/ja4_fingerprint/ja4.h
+++ b/plugins/experimental/ja4_fingerprint/ja4.h
@@ -54,7 +54,7 @@ public:
   using difference_type = 
std::iterator_traits<std::vector<std::uint16_t>::iterator>::difference_type;
 
   Protocol      protocol;
-  std::uint16_t TLS_version;
+  std::uint16_t TLS_version{0}; // 0 is not the default, this is only to not 
have it un-initialized.
   std::string   ALPN;
 
   std::vector<std::uint16_t> const &get_ciphers() const;

Reply via email to