commit 65be834c30fb43abb2e41585b41eefcd2ae06c01
author h-east <h.east.727@gmail.com> Sun Dec 08 10:05:26 2024 +0100
committer Christian Brabandt <cb@256bit.org> Sun Dec 08 10:08:48 2024 +0100
tree 28ecfe18de5e1e0ee80ad07173b2ac075319c701
parent 92b36663f8d0e507f60f357c6add6f6c9148a951

patch 9.1.0913: no error check for neg values for 'messagesopt'

Problem:  no error check for neg values for 'messagesopt'
          (after v9.1.0908)
Solution: add additional error checks and tests (h-east)

closes: #16187

Signed-off-by: Shougo Matsushita <Shougo.Matsu@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>

src/message.c

diff --git a/src/message.c b/src/message.c
index f0d1806..0fb0013 100644
--- a/src/message.c
+++ b/src/message.c
@@ -1133,12 +1133,10 @@
     if (!(messages_flags_new & MESSAGES_HISTORY))
         return FAIL;
 
-    // "history" must be <= 10000
-    if (messages_history_new > 10000)
+    if (messages_history_new < 0 || messages_history_new > 10000)
         return FAIL;
 
-    // "wait" must be <= 10000
-    if (messages_wait_new > 10000)
+    if (messages_wait_new < 0 || messages_wait_new > 10000)
         return FAIL;
 
     msg_flags = messages_flags_new;