awcullen / opcua

OPC Unified Architecture (OPC UA) in Go.
MIT License
81 stars 18 forks source link

fix: remove diagnostics nodes for expired sessions #8

Closed milgner closed 1 year ago

milgner commented 2 years ago

When a session expired, the corresponding diagnostics node would remain in the tree. Not only would this clutter up the tree, but it might also allow for a DoS-type scenario which bypasses the MaxSessionCount restriction where an attacker could open sessions with low timeout, causing uncontrolled memory growth due to orphaned diagnostics nodes.

awcullen commented 1 year ago

Thanks very much @milgner