From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Fri, 29 Jul 2016 15:30:31 +0000 (+0100)
Subject: [Fix] Do not break history on NaN in required score
X-Git-Tag: 1.3.1~16
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=921fb54d87814c584916b469dc1b83a779587098;p=thirdparty%2Frspamd.git

[Fix] Do not break history on NaN in required score

Issue: #763
Reported by: @fatalbanana
---

diff --git a/src/controller.c b/src/controller.c
index 103f97be5b..394110d0ad 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 (