From 18d2f273b506895bf435a40e37f17ed4afafcd7e Mon Sep 17 00:00:00 2001 From: "Joshua C. Colp" Date: Mon, 25 Apr 2022 20:39:10 -0300 Subject: [PATCH] manager: Terminate session on write error. On a write error to an AMI session a flag was set to indicate that the write error had occurred, with the expected result being that the session be terminated. This was not actually happening and instead writing would continue to be attempted. This change adds a check for the write error and causes the session to actually terminate. ASTERISK-29948 Change-Id: Icaf5d413d4c0d5dc78292a17287fecc8720a31a5 --- main/manager.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main/manager.c b/main/manager.c index 76a6611a24..4c55823a77 100644 --- a/main/manager.c +++ b/main/manager.c @@ -6242,7 +6242,7 @@ static int process_events(struct mansession *s) (s->session->readperm & eqe->category) == eqe->category && (s->session->send_events & eqe->category) == eqe->category) { if (match_filter(s, eqe->eventdata)) { - if (send_string(s, eqe->eventdata) < 0) + if (send_string(s, eqe->eventdata) < 0 || s->write_error) ret = -1; /* don't send more */ } } -- 2.47.2