30 Mar
2025
30 Mar
'25
5:50 p.m.
@vingarzan commented on this pull request.
@@ -1351,8 +1347,13 @@ void Rcv_Process(peer *p, AAAMessage *msg)
return; }
- if(msg->sessionId) + if(msg->sessionId) { + // Ensure proper locking order + lock_release(p->lock);
yup, I agree... it's the same thing as removed from `routing.c`, just that the peer and the session are swapped. Removing that is a good idea (but might break other things).
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4191#discussion_r2020216919
You are receiving this because you are subscribed to this thread.
Message ID: kamailio/kamailio/pull/4191/review/2727971087@github.com