patch 8.2.3589: failure when "term_rows" of term_start() is an unusual value

Problem:    Failure when the "term_rows" argument of term_start() is an
            unusual value.
Solution:   Limit to range of zero to 1000. (closes #9116)
diff --git a/runtime/doc/terminal.txt b/runtime/doc/terminal.txt
index 510f1c3..47b04df 100644
--- a/runtime/doc/terminal.txt
+++ b/runtime/doc/terminal.txt
@@ -871,7 +871,8 @@
 		   "term_name"	     name to use for the buffer name, instead
 				     of the command name.
 		   "term_rows"	     vertical size to use for the terminal,
-				     instead of using 'termwinsize'
+				     instead of using 'termwinsize'; valid
+				     range is from zero to 1000
 		   "term_cols"	     horizontal size to use for the terminal,
 				     instead of using 'termwinsize'
 		   "vertical"	     split the window vertically; note that
diff --git a/src/job.c b/src/job.c
index 80cb47e..a2e184b 100644
--- a/src/job.c
+++ b/src/job.c
@@ -432,6 +432,11 @@
 		opt->jo_term_rows = tv_get_number_chk(item, &error);
 		if (error)
 		    return FAIL;
+		if (opt->jo_term_rows < 0 || opt->jo_term_rows > 1000)
+		{
+		    semsg(_(e_invargval), "term_rows");
+		    return FAIL;
+		}
 	    }
 	    else if (STRCMP(hi->hi_key, "term_cols") == 0)
 	    {
diff --git a/src/testdir/test_terminal.vim b/src/testdir/test_terminal.vim
index 3e263ef..cd7f477 100644
--- a/src/testdir/test_terminal.vim
+++ b/src/testdir/test_terminal.vim
@@ -467,6 +467,8 @@
   bwipe!
   call assert_equal([7, 27], size)
 
+  call assert_fails("call term_start(cmd, {'term_rows': -1})", 'E475:')
+  call assert_fails("call term_start(cmd, {'term_rows': 1001})", 'E475:')
   if has('float')
     call assert_fails("call term_start(cmd, {'term_rows': 10.0})", 'E805:')
   endif
diff --git a/src/version.c b/src/version.c
index 4be3fd2..8e2a10b 100644
--- a/src/version.c
+++ b/src/version.c
@@ -758,6 +758,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    3589,
+/**/
     3588,
 /**/
     3587,