[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor] branch main updated: Coverity CID 1518991: Tighter bounds on consensus param value.



This is an automated email from the git hooks/post-receive script.

dgoulet pushed a commit to branch main
in repository tor.

The following commit(s) were added to refs/heads/main by this push:
     new c6ef04e0d0 Coverity CID 1518991: Tighter bounds on consensus param value.
     new 754dbcd6d3 Merge branch 'maint-0.4.7'
c6ef04e0d0 is described below

commit c6ef04e0d07418f447209d86d642394a190b6702
Author: Mike Perry <mikeperry-git@xxxxxxxxxxxxxx>
AuthorDate: Wed Jan 11 17:32:20 2023 +0000

    Coverity CID 1518991: Tighter bounds on consensus param value.
    
    This prevents sign extension overflow in cwnd_became_full().
---
 src/core/or/congestion_control_vegas.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/core/or/congestion_control_vegas.c b/src/core/or/congestion_control_vegas.c
index 8f9c39cea6..b82c685d51 100644
--- a/src/core/or/congestion_control_vegas.c
+++ b/src/core/or/congestion_control_vegas.c
@@ -203,7 +203,7 @@ congestion_control_vegas_set_params(congestion_control_t *cc,
    networkstatus_get_param(NULL, "cc_cwnd_full_gap",
       VEGAS_CWND_FULL_GAP_DFLT,
       0,
-      INT32_MAX);
+      INT16_MAX);
 
   cc_cwnd_full_per_cwnd =
    networkstatus_get_param(NULL, "cc_cwnd_full_per_cwnd",

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits