@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/2727971...@github.com>_______________________________________________
Kamailio - Development Mailing List -- sr-dev@lists.kamailio.org
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
Important: keep the mailing list in the recipients, do not reply only to the
sender!
