devs/cproof
1
2
Fork 1
Code Issues 43 Pull Requests 8 Actions Packages Projects 1 Releases Wiki Activity

Merge pull request #1633 from profanity-im/fix-split-url-once-more

Browse Source 
Fix `_split_url()` once more
This commit is contained in:
Michael Vetter
2022-02-02 10:22:04 +01:00
committed by GitHub
parent 832f28bcb2 569525c6c7
commit e0f75e0b52
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/xmpp/connection.c
View File

@@ -935,7 +935,7 @@ _split_url(const char* alturi, gchar** host, gint* port)
char* last = strrchr(alturi, ':'); char* last = strrchr(alturi, ':');
if (first) { if (first) {
if (first == last) { if (first == last) {
hostlen = last - alturi; hostlen = last - alturi + 1;
if (!strtoi_range(last + 1, port, 1, 65535, NULL)) if (!strtoi_range(last + 1, port, 1, 65535, NULL))
return FALSE; return FALSE;
} else { } else {
@@ -949,7 +949,7 @@ _split_url(const char* alturi, gchar** host, gint* port)
*port = 0; *port = 0;
} else { } else {
/* `[ip:v6]:port` */ /* `[ip:v6]:port` */
hostlen = last - alturi; hostlen = last - alturi + 1;
if (!strtoi_range(last + 1, port, 1, 65535, NULL)) if (!strtoi_range(last + 1, port, 1, 65535, NULL))
return FALSE; return FALSE;
} }