[SLING-1728] ServiceCache is prone to deadlocks

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: Scripting Core 2.0.14
• Fix Version/s: Scripting Core 2.0.14
• Component/s: Scripting
• Assignee: Felix Meschberger
• Reporter: Felix Meschberger
• Created: 03/Sep/10 17:05
• Updated: 02/Nov/10 13:21
• Resolved: 06/Sep/10 06:14

Description

The ServiceCache introduce with SLING-1478 is prone to deadlocks with the Framework.

The serviceChanged method is called from the framework and tries to synchronize on the cache. The getService method on the other hand synchronizes on the cache and while holding the lock calls into the framework.

The fix is to not hold a lock while calling into the framework.