edg-l/ddnet
1
0
Fork 0
mirror of https://github.com/ddnet/ddnet.git synced 2024-11-10 01:58:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge #5285

Browse source 
5285: Fix clamp in CStun::Update (fixes #5282) r=heinrich5991 a=def-

<!-- What is the motivation for the changes of this pull request -->

## Checklist

- [ ] Tested the change ingame
- [ ] Provided screenshots if it is a visual change
- [ ] Tested in combination with possibly related configuration options
- [ ] Written a unit test if it works standalone, system.c especially
- [ ] Considered possible null pointers and out of bounds array indexing
- [ ] Changed no physics that affect existing maps
- [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--undefinedbehavioursanitizer-or-valgrinds-memcheck) (optional)


Co-authored-by: Dennis Felsing <dennis@felsin9.de>
This commit is contained in:
bors[bot] 2022-05-31 12:11:24 +00:00 committed by GitHub
parent 4183ad6b47 e31fa7051e
commit 369c217db5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/engine/shared/network_stun.cpp
View file

@ -30,7 +30,7 @@ static const char *IndexToSystem(int Index)
static int RetryWaitSeconds(int NumUnsuccessfulTries)
{
return (1 << clamp(0, NumUnsuccessfulTries, 9));
return (1 << clamp(NumUnsuccessfulTries, 0, 9));
}
CStun::CProtocol::CProtocol(int Index, NETSOCKET Socket) :