add_saturation approach in calculateExpirationTime

ErakhtinB · ErakhtinB · commit 59bde8d60b3f · 2025-04-10T14:48:37.000+05:00
diff --git a/src/peer/address_repository/inmem_address_repository.cpp b/src/peer/address_repository/inmem_address_repository.cpp
@@ -66,11 +66,11 @@ namespace libp2p::peer {
 
   Clock::time_point InmemAddressRepository::calculateExpirationTime(
       const Milliseconds &ttl) const {
-    if (ttl >= std::chrono::milliseconds::max() / 2) {
-      // Handle "permanent" or extremely long TTL values safely
+    auto now = Clock::now();
+    if (now >= Clock::time_point::max() - ttl) {
       return Clock::time_point::max();
     }
-    return Clock::now() + ttl;
+    return now + ttl;
   }
 
   outcome::result<bool> InmemAddressRepository::addAddresses(

Original file line number	Diff line number	Diff line change
`@@ -66,11 +66,11 @@ namespace libp2p::peer {`
`66`	`66`
`67`	`67`	`Clock::time_point InmemAddressRepository::calculateExpirationTime(`
`68`	`68`	`const Milliseconds &ttl) const {`
`69`		`- if (ttl >= std::chrono::milliseconds::max() / 2) {`
`70`		`- // Handle "permanent" or extremely long TTL values safely`
	`69`	`+ auto now = Clock::now();`
	`70`	`+ if (now >= Clock::time_point::max() - ttl) {`
`71`	`71`	`return Clock::time_point::max();`
`72`	`72`	`}`
`73`		`- return Clock::now() + ttl;`
	`73`	`+ return now + ttl;`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`outcome::result<bool> InmemAddressRepository::addAddresses(`