fix thread arguments limit

docs.md

@@ -73,7 +73,7 @@ used here to facilitate documenting consistent behavior:
  metamethod
 - `buffer`: a `string` or a sequential `table` of `string`s
 - `threadargs`: variable arguments (`...`) of type `nil`, `boolean`, `number`,
- `string`, or `userdata`
+ `string`, or `userdata`, numbers of argument limited to 9.
 
 ## Contents
 
@@ -3271,6 +3271,8 @@ provided. Currently accepted `option` fields are `stack_size`.
 
 **Returns:** `luv_thread_t userdata` or `fail`
 
+**Note:** unsafe, please make sure the thread end of life before the Lua state close.
+
 ### `uv.thread_equal(thread, other_thread)`
 
 > method form `thread:equal(other_thread)`

src/thread.c

@@ -65,7 +65,7 @@ static int luv_thread_arg_set(lua_State* L, luv_thread_arg_t* args, int idx, int
  idx = idx > 0 ? idx : 1;
  i = idx;
  args->flags = flags;
- while (i <= top && i <= LUV_THREAD_MAXNUM_ARG + idx)
+ while (i <= top && i < LUV_THREAD_MAXNUM_ARG + idx)
  {
  luv_val_t *arg = args->argv + i - idx;
  arg->type = lua_type(L, i);

tests/test-thread.lua

@@ -82,4 +82,21 @@ return require('lib/tap')(function (test)
  assert(elapsed >= 100, "elapsed should be at least delay ")
  end, "1.26.0")
 
+ test("test thread arguments limit", function(print, p, expect, uv)
+ local args = {}
+ args[1] = uv.new_async(expect(function (n)
+ assert(n==9)
+ args[1]:close()
+ end))
+ for i=2, 10 do
+ args[i] = i
+ end
+ local unpack = unpack or table.unpack
+ uv.new_thread(function(...)
+ local arg = {...}
+ assert(#arg == 9)
+ arg[1]:send(#arg)
+ end, unpack(args)):join()
+ assert(#args==10)
+ end)
 end)