fix major issues from sonarqube

[cogross]

• Interrupted exceptions should be rethrown OR method should be re-interrupted (see https://cwe.mitre.org/data/definitions/391)
• Method parameters should not be used for synchronization (see MITRE, CWE-412 - Unrestricted Externally Accessible Lock | MITRE, CWE-413 - Improper Resource Locking | CERT, LCK00-J. - Use private final lock objects to synchronize classes that may interact with untrusted code)
• Synchronization should be done on instances of value-based classes (see http://localhost:9000/coding_rules?open=java%3AS1860&rule_key=java%3AS1860)