Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@vingarzan commented on this pull request.
In src/modules/cdp/peerstatemachine.c:
> @@ -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, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.