DEBUG: stream-int: Fix BUG_ON used to test appctx in si_applet_ops callbacks
693b23bb1 ("MEDIUM: tree-wide: Use unsafe conn-stream API when it is
relevant") introduced a regression in DEBUG_STRICT mode because some BUG_ON
conditions were inverted. It should ok now.
In addition, ALREADY_CHECKED macro was removed from appctx_wakeup() function
because it is useless now.
diff --git a/src/stream_interface.c b/src/stream_interface.c
index c5112df..35f66d7 100644
--- a/src/stream_interface.c
+++ b/src/stream_interface.c
@@ -1656,7 +1656,7 @@
{
struct channel *ic = si_ic(si);
- BUG_ON(cs_appctx(si->cs));
+ BUG_ON(!cs_appctx(si->cs));
/* If the applet wants to write and the channel is closed, it's a
* broken pipe and it must be reported.
@@ -1696,7 +1696,7 @@
{
struct channel *ic = si_ic(si);
- BUG_ON(cs_appctx(si->cs));
+ BUG_ON(!cs_appctx(si->cs));
si_rx_shut_blk(si);
if (ic->flags & CF_SHUTR)
@@ -1732,7 +1732,7 @@
struct channel *ic = si_ic(si);
struct channel *oc = si_oc(si);
- BUG_ON(cs_appctx(si->cs));
+ BUG_ON(!cs_appctx(si->cs));
oc->flags &= ~CF_SHUTW_NOW;
if (oc->flags & CF_SHUTW)
@@ -1785,7 +1785,7 @@
{
struct channel *ic = si_ic(si);
- BUG_ON(cs_appctx(si->cs));
+ BUG_ON(!cs_appctx(si->cs));
DPRINTF(stderr, "%s: si=%p, si->state=%d ic->flags=%08x oc->flags=%08x\n",
__FUNCTION__,
@@ -1802,7 +1802,7 @@
{
struct channel *oc = si_oc(si);
- BUG_ON(cs_appctx(si->cs));
+ BUG_ON(!cs_appctx(si->cs));
DPRINTF(stderr, "%s: si=%p, si->state=%d ic->flags=%08x oc->flags=%08x\n",
__FUNCTION__,