MINOR: mux-h2: always use h2_send() in h2_done_ff(), not h2_process()
authorWilly Tarreau <w@1wt.eu>
Sat, 4 Nov 2023 07:12:17 +0000 (08:12 +0100)
committerWilly Tarreau <w@1wt.eu>
Sat, 4 Nov 2023 07:12:17 +0000 (08:12 +0100)
commit0fa5adee3bb17814b80ddfb5e07f5e219efcfebd
treeaca38db9d7566aa11d82207966940599b41a1e28
parent58185669d8a0891948caaa7b357d78775b0cecb3
MINOR: mux-h2: always use h2_send() in h2_done_ff(), not h2_process()

By calling h2_process(), the code would theoretically make it possible
for a synchronous ->wake() call to provoke an indirect call to h2_snd_buf()
while we're in h2_done_ff(), which could be quite bad. The current
conditions do not permit it right now but this could easily break by
accident. Better use h2_send() and wake the task up if needed. Precise
performance tests showed no change.
src/mux_h2.c