You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bn...@apache.org on 2023/09/27 16:39:47 UTC

[trafficserver] branch master updated: NextHopHealthStatus: CID 1518604: Use of 32-bit time_t (#10536)

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

bnolsen 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 0284df6d80 NextHopHealthStatus: CID 1518604: Use of 32-bit time_t (#10536)
0284df6d80 is described below

commit 0284df6d8052a3edd71579df565caf247bbc329d
Author: Brian Olsen <br...@comcast.com>
AuthorDate: Wed Sep 27 10:39:41 2023 -0600

    NextHopHealthStatus: CID 1518604: Use of 32-bit time_t (#10536)
---
 proxy/http/remap/NextHopHealthStatus.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http/remap/NextHopHealthStatus.cc b/proxy/http/remap/NextHopHealthStatus.cc
index 7cff756187..9d5b16d805 100644
--- a/proxy/http/remap/NextHopHealthStatus.cc
+++ b/proxy/http/remap/NextHopHealthStatus.cc
@@ -131,7 +131,7 @@ NextHopHealthStatus::markNextHop(TSHttpTxn txn, const char *hostname, const int
       // if the last failure was outside the retry window, set the failcount to 1 and failedAt to now.
       { // lock guard
         std::lock_guard<std::mutex> lock(h->_mutex);
-        if ((h->failedAt.load() + retry_time) < static_cast<unsigned>(_now)) {
+        if ((h->failedAt.load() + retry_time) < _now) {
           new_fail_count = h->failCount = 1;
           h->failedAt                   = _now;
         } else {