diff options
| author | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2016-07-29 16:30:31 +0100 |
|---|---|---|
| committer | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2016-07-29 16:32:01 +0100 |
| commit | 921fb54d87814c584916b469dc1b83a779587098 (patch) | |
| tree | e9b2bac096af3082d886882646a89e6898cb25f8 | |
| parent | 0ce4ffeb9f85cb412116f4fe008c72961c047fe0 (diff) | |
| download | rspamd-921fb54d87814c584916b469dc1b83a779587098.tar.gz rspamd-921fb54d87814c584916b469dc1b83a779587098.zip | |
[Fix] Do not break history on NaN in required score
Issue: #763
Reported by: @fatalbanana
| -rw-r--r-- | src/controller.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/controller.c b/src/controller.c index 103f97be5..394110d0a 100644 --- a/src/controller.c +++ b/src/controller.c @@ -1277,9 +1277,17 @@ rspamd_controller_handle_history (struct rspamd_http_connection_entry *conn_ent, row->action)), "action", 0, false); ucl_object_insert_key (obj, ucl_object_fromdouble ( row->score), "score", 0, false); - ucl_object_insert_key (obj, - ucl_object_fromdouble ( - row->required_score), "required_score", 0, false); + + if (!isnan (row->required_score)) { + ucl_object_insert_key (obj, + ucl_object_fromdouble ( + row->required_score), "required_score", 0, false); + } + else { + ucl_object_insert_key (obj, + ucl_object_fromdouble (0.0), "required_score", 0, false); + } + ucl_object_insert_key (obj, ucl_object_fromstring ( row->symbols), "symbols", 0, false); ucl_object_insert_key (obj, ucl_object_fromint ( |