BUG/MINOR: Fix the sending function in Lua's cosocket
This is a regression from the commit a73e59b6901a164d19b1145e8511602d9814f28f.
When data are sent from a cosocket, the action is done in the context of the
applet running a lua stack and not in the context of the applet owning the
cosocket. So we must take care, explicitly, that this last applet have a buffer
(the req buffer of the cosocket).
This patch must be backported in 1.7
diff --git a/src/hlua.c b/src/hlua.c
index f662cb0..17a6750 100644
--- a/src/hlua.c
+++ b/src/hlua.c
@@ -1889,14 +1889,19 @@
* the request buffer if its not required.
*/
if (socket->s->req.buf->size == 0) {
- si_applet_cant_put(&socket->s->si[0]);
- goto hlua_socket_write_yield_return;
+ appctx = hlua->task->context;
+ if (!channel_alloc_buffer(&socket->s->req, &appctx->buffer_wait))
+ goto hlua_socket_write_yield_return;
}
/* Check for avalaible space. */
len = buffer_total_space(socket->s->req.buf);
- if (len <= 0)
+ if (len <= 0) {
+ appctx = objt_appctx(socket->s->si[0].end);
+ if (!hlua_com_new(&hlua->com, &appctx->ctx.hlua_cosocket.wake_on_write, hlua->task))
+ WILL_LJMP(luaL_error(L, "out of memory"));
goto hlua_socket_write_yield_return;
+ }
/* send data */
if (len < send_len)
@@ -1934,9 +1939,6 @@
return 1;
hlua_socket_write_yield_return:
- appctx = objt_appctx(socket->s->si[0].end);
- if (!hlua_com_new(&hlua->com, &appctx->ctx.hlua_cosocket.wake_on_write, hlua->task))
- WILL_LJMP(luaL_error(L, "out of memory"));
WILL_LJMP(hlua_yieldk(L, 0, 0, hlua_socket_write_yield, TICK_ETERNITY, 0));
return 0;
}