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

Merge pull request #8955 from furo321/url-master-parsing

Browse source 
Don't allow URLs without port from the masterserver
This commit is contained in:
Dennis Felsing 2024-09-15 09:28:43 +00:00 committed by GitHub
parent dcd02b50bb 7bbd51cf73
commit 4b6f2e22a8
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/engine/client/serverbrowser_http.cpp
View file

@ -424,7 +424,10 @@ void CServerBrowserHttp::Refresh()
} }
bool ServerbrowserParseUrl(NETADDR *pOut, const char *pUrl) bool ServerbrowserParseUrl(NETADDR *pOut, const char *pUrl)
{ {
return net_addr_from_url(pOut, pUrl, nullptr, 0) != 0; int Failure = net_addr_from_url(pOut, pUrl, nullptr, 0);
if(Failure || pOut->port == 0)
return true;
return false;
} }
bool CServerBrowserHttp::Validate(json_value *pJson) bool CServerBrowserHttp::Validate(json_value *pJson)
{ {