aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [Minor] do not pass invalid C warnings to C++ flagsChristian Göttsche2020-01-161-2/+4
| | | | | cc1plus: warning: command line option ‘-Wno-pointer-sign’ is valid for C/ObjC but not for C++ cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
* [Minor] use modern name -Wextra instead of -WChristian Göttsche2020-01-161-5/+5
|
* [Minor] silence redefinition of macroChristian Göttsche2020-01-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [74/340] Building C object contrib/lua-lpeg/CMakeFiles/rspamd-lpeg.dir/lpprint.c.o In file included from ../contrib/lua-lpeg/lpprint.c:11: ../contrib/lua-lpeg/lptypes.h:15: warning: "NDEBUG" redefined 15 | #define NDEBUG | <command-line>: note: this is the location of the previous definition [75/340] Building C object contrib/lua-lpeg/CMakeFiles/rspamd-lpeg.dir/lpcap.c.o In file included from ../contrib/lua-lpeg/lpcap.h:9, from ../contrib/lua-lpeg/lpcap.c:9: ../contrib/lua-lpeg/lptypes.h:15: warning: "NDEBUG" redefined 15 | #define NDEBUG | <command-line>: note: this is the location of the previous definition [77/340] Building C object contrib/lua-lpeg/CMakeFiles/rspamd-lpeg.dir/lpvm.c.o In file included from ../contrib/lua-lpeg/lpcap.h:9, from ../contrib/lua-lpeg/lpvm.c:15: ../contrib/lua-lpeg/lptypes.h:15: warning: "NDEBUG" redefined 15 | #define NDEBUG | <command-line>: note: this is the location of the previous definition [79/340] Building C object contrib/lua-lpeg/CMakeFiles/rspamd-lpeg.dir/lpcode.c.o In file included from ../contrib/lua-lpeg/lpcode.c:12: ../contrib/lua-lpeg/lptypes.h:15: warning: "NDEBUG" redefined 15 | #define NDEBUG | <command-line>: note: this is the location of the previous definition [81/340] Building C object contrib/lua-lpeg/CMakeFiles/rspamd-lpeg.dir/lptree.c.o In file included from ../contrib/lua-lpeg/lptree.c:15: ../contrib/lua-lpeg/lptypes.h:15: warning: "NDEBUG" redefined 15 | #define NDEBUG | <command-line>: note: this is the location of the previous definition
* [Minor] silence -Wmisleading-indentation in contrib/snowball/Christian Göttsche2020-01-161-4/+3
| | | | | | | | | | | | | | | | | | | | | | | ../contrib/snowball/compiler/analyser.c: In function ‘check_name_type’: ../contrib/snowball/compiler/analyser.c:210:19: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] 210 | case 'r': if (p->type == t_routine || | ^~ ../contrib/snowball/compiler/analyser.c:211:54: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’ 211 | p->type == t_external) return; break; | ^~~~~ ../contrib/snowball/compiler/analyser.c: In function ‘read_program’: ../contrib/snowball/compiler/analyser.c:859:21: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] 859 | if (q->used && q->definition == 0) error4(a, q); break; | ^~ ../contrib/snowball/compiler/analyser.c:859:70: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’ 859 | if (q->used && q->definition == 0) error4(a, q); break; | ^~~~~ ../contrib/snowball/compiler/analyser.c:861:21: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] 861 | if (q->used && q->grouping == 0) error4(a, q); break; | ^~ ../contrib/snowball/compiler/analyser.c:861:68: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’ 861 | if (q->used && q->grouping == 0) error4(a, q); break; | ^~~~~
* [Minor] Lua_task: Remove unused crapVsevolod Stakhov2020-01-161-23/+6
| | | | Issue: #3220
* Merge pull request #3219 from cgzones/interface_cruftVsevolod Stakhov2020-01-162-20/+0
|\ | | | | [Minor] remove cruft files from webui
| * [Minor] remove cruft files from webuiChristian Göttsche2020-01-162-20/+0
| |
* | Merge pull request #3218 from cgzones/no_replxxVsevolod Stakhov2020-01-161-0/+6
|\ \ | | | | | | [Minor] fix compilation without replxx
| * | [Minor] fix compilation without replxxChristian Göttsche2020-01-161-0/+6
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ../src/rspamadm/lua_repl.c: In function ‘rspamadm_lua_run_repl’: ../src/rspamadm/lua_repl.c:615:8: warning: unused variable ‘i’ [-Wunused-variable] 615 | gsize i; | ^ ../src/rspamadm/lua_repl.c:614:11: warning: unused variable ‘tb’ [-Wunused-variable] 614 | GString *tb = NULL; | ^~ ../src/rspamadm/lua_repl.c:613:11: warning: unused variable ‘is_multiline’ [-Wunused-variable] 613 | gboolean is_multiline = FALSE; | ^~~~~~~~~~~~ ../src/rspamadm/lua_repl.c: In function ‘rspamadm_lua’: ../src/rspamadm/lua_repl.c:1007:3: warning: implicit declaration of function ‘replxx_set_max_history_size’ [-Wimplicit-function-declaration] 1007 | replxx_set_max_history_size (rx_instance, max_history); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ ../src/rspamadm/lua_repl.c:1007:32: error: ‘rx_instance’ undeclared (first use in this function) 1007 | replxx_set_max_history_size (rx_instance, max_history); | ^~~~~~~~~~~ ../src/rspamadm/lua_repl.c:1007:32: note: each undeclared identifier is reported only once for each function it appears in ../src/rspamadm/lua_repl.c:1008:3: warning: implicit declaration of function ‘replxx_history_load’ [-Wimplicit-function-declaration] 1008 | replxx_history_load (rx_instance, histfile); | ^~~~~~~~~~~~~~~~~~~ ../src/rspamadm/lua_repl.c:1010:3: warning: implicit declaration of function ‘replxx_history_save’ [-Wimplicit-function-declaration] 1010 | replxx_history_save (rx_instance, histfile); | ^~~~~~~~~~~~~~~~~~~
* | Merge pull request #3217 from cgzones/exampledirVsevolod Stakhov2020-01-163-8/+1
|\ \ | | | | | | [Minor] remove unused examplesdir
| * | [Minor] remove unused examplesdirChristian Göttsche2020-01-163-8/+1
| |/
* | Merge pull request #3216 from cgzones/spellingsVsevolod Stakhov2020-01-161-1/+1
|\ \ | |/ |/| [Minor] fix spelling
| * [Minor] fix spellingChristian Göttsche2020-01-161-1/+1
|/
* [Minor] Check string length to avoid OOB readingVsevolod Stakhov2020-01-161-1/+1
|
* Merge pull request #3153 from resec/smembersVsevolod Stakhov2020-01-161-29/+47
|\ | | | | [Minor] Eliminate redis SMEMBERS usage
| * eliminate smembers: remove redundent cleanupresec2019-11-191-11/+1
| |
| * eliminate smembers: fix againresec2019-11-191-5/+12
| |
| * eliminate smembers: fixresec2019-11-191-6/+5
| |
| * eliminate redis smembersresec2019-11-191-29/+51
| |
* | Merge pull request #3214 from Lekensteyn/fix-dkim-policy-checkVsevolod Stakhov2020-01-161-3/+3
|\ \ | | | | | | [Minor] Dkim_signing: correct is_skip_sign logic
| * | [Minor] Dkim_signing: correct is_skip_sign logicPeter Wu2020-01-161-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If any of "sign_networks", "auth_only", or "sign_local" are disabled, then it should not automatically proceed with signing if the enabled conditions all fail. For example, if only the auth_only setting is enabled, and is_authed is false, then signing should be skipped. An earlier check luckily prevents this correctness issue from being exploitable ("ignoring unauthenticated mail"), but fix the logic anyway.
* | | [Project] Lua_content: Add preliminary fonts handlingVsevolod Stakhov2020-01-161-52/+75
| | |
* | | [Minor] Try to fix slow timer race conditionVsevolod Stakhov2020-01-161-5/+22
|/ /
* | [Fix] Fix urls shifting when doing decode to include separatorsVsevolod Stakhov2020-01-152-4/+4
| |
* | Merge pull request #3213 from citrin/url-frag-testVsevolod Stakhov2020-01-151-0/+3
|\ \ | | | | | | [Test] Add a test case for URL parsing
| * | [Test] Add a test case for URL parsingAnton Yuzhaninov2020-01-141-0/+3
| | | | | | | | | | | | Currently it fails because fragment is not extracted correctly.
* | | [Minor] Just another try to avoid raceVsevolod Stakhov2020-01-151-6/+11
|/ /
* | [Project] Lua_content: Implement url's extraction from PDF filesVsevolod Stakhov2020-01-141-0/+36
| |
* | [Project] Lua_content: Implement ObjStm decodingVsevolod Stakhov2020-01-141-42/+75
| |
* | Merge pull request #3212 from moisseev/patch-1Vsevolod Stakhov2020-01-141-1/+1
|\ \ | | | | | | [Minor] bayes_expiry: Fix type check (https://github.com/antirez/redis/issues/3231)
| * | [Minor] bayes_expiry: Fix type checkAlexander Moisseev2020-01-141-1/+1
|/ /
* | [Project] Lua_content: Add preliminary support of compound objectsVsevolod Stakhov2020-01-131-48/+75
| |
* | [Minor] Lua_text: Set class properlyVsevolod Stakhov2020-01-131-0/+1
| |
* | [Project] Lua_content: Massive rework of the parsing structureVsevolod Stakhov2020-01-131-76/+115
| |
* | [Minor] Lua_content: Similar hang fixVsevolod Stakhov2020-01-131-1/+1
| |
* | [Minor] Lua_content: Fix some more crappy pdf issuesVsevolod Stakhov2020-01-131-2/+2
| |
* | [Minor] Lua_content: Fix hangVsevolod Stakhov2020-01-131-2/+5
| |
* | [Fix] Fix urls encode functionVsevolod Stakhov2020-01-131-2/+2
| |
* | [Fix] Avoid double escapingVsevolod Stakhov2020-01-131-5/+5
| |
* | [Fix] Fix dealing with `\0` in ucl strings and JSONVsevolod Stakhov2020-01-133-1/+13
| |
* | [Minor] Another try to fix raceVsevolod Stakhov2020-01-131-0/+1
| |
* | [Minor] Sigh - another try to avoid racesVsevolod Stakhov2020-01-131-4/+4
| |
* | [Minor] Bayes_expiry: Test type before trying to evaluateVsevolod Stakhov2020-01-131-17/+20
| | | | | | | | Issue: #3208
* | Merge pull request #3210 from moisseev/spamtrapVsevolod Stakhov2020-01-121-1/+1
|\ \ | | | | | | [Conf] Update spamtrap map path example
| * | [Conf] Update spamtrap map path examplemoisseev2020-01-121-1/+1
|/ /
* | [Minor] Rbl: Fix resolve ip options insertionVsevolod Stakhov2020-01-111-9/+18
| |
* | Merge pull request #3206 from korgoth1/masterVsevolod Stakhov2020-01-111-0/+6
|\ \ | | | | | | [Test] SURBL resolve ip
| * | [Test] SURBL resolve ipkorgoth12020-01-111-3/+3
| | |
| * | [Test] SURBL resolve ipkorgoth12020-01-101-1/+1
| | |
| * | [Test] SURBL resolve ipkorgoth12020-01-1013-71/+160
| |\ \