diff options
author | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2021-04-15 21:23:43 +0100 |
---|---|---|
committer | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2021-04-15 21:23:43 +0100 |
commit | f09b8e5be2aa8ad1593bcb0e7438af6b6d5a1305 (patch) | |
tree | c657c43710622cdc801690fb62657ec02223ac79 /src | |
parent | 852f2c7f209507fe0a80da9a9b1c43f610179fb5 (diff) | |
download | rspamd-f09b8e5be2aa8ad1593bcb0e7438af6b6d5a1305.tar.gz rspamd-f09b8e5be2aa8ad1593bcb0e7438af6b6d5a1305.zip |
[Minor] Avoid FP when a protocol prefix is implicitly added
Diffstat (limited to 'src')
-rw-r--r-- | src/libserver/html.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/libserver/html.c b/src/libserver/html.c index 5e29b3b84..137907183 100644 --- a/src/libserver/html.c +++ b/src/libserver/html.c @@ -837,8 +837,8 @@ rspamd_html_url_is_phished (rspamd_mempool_t *pool, */ gboolean obfuscation_found = FALSE; - if (g_ascii_strncasecmp (url_str, "http", 4) == 0 && - strstr (url_str, "://") != NULL) { + if (len > 4 && g_ascii_strncasecmp (url_text, "http", 4) == 0 && + rspamd_substring_search (url_text, len,"://", 3) != -1) { /* Clearly an obfuscation attempt */ obfuscation_found = TRUE; } |