BUG/MINOR: lua funtion hlua_socket_settimeout don't check negative values

Negatives timeouts doesn't have sense. A negative timeout doesn't cause
a crash, but the connection expires before the system try to extablish it.

This patch should be backported in all versions from 1.6
This commit is contained in:
Thierry Fournier 2018-03-08 09:59:02 +01:00 committed by Willy Tarreau
parent e9636f192a
commit 17a921b799

View File

@ -2474,6 +2474,10 @@ __LJMP static int hlua_socket_settimeout(struct lua_State *L)
socket = MAY_LJMP(hlua_checksocket(L, 1));
tmout = MAY_LJMP(luaL_checkinteger(L, 2)) * 1000;
/* Check for negative values */
if (tmout < 0)
WILL_LJMP(luaL_error(L, "settimeout: cannot set negatives values"));
/* Check if we run on the same thread than the xreator thread.
* We cannot access to the socket if the thread is different.
*/