You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by da...@apache.org on 2015/06/10 12:32:46 UTC
svn commit: r1684635 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java
Author: davidb
Date: Wed Jun 10 10:32:46 2015
New Revision: 1684635
URL: http://svn.apache.org/r1684635
Log:
Add defensive null check.
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java
Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java?rev=1684635&r1=1684634&r2=1684635&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/ServiceRegistry.java Wed Jun 10 10:32:46 2015
@@ -419,7 +419,8 @@ public class ServiceRegistry
{
if (count == 0)
{
- Object svc = usage.m_svcHolderRef.getAndSet(null).m_service;
+ ServiceHolder holder = usage.m_svcHolderRef.getAndSet(null);
+ Object svc = holder != null ? holder.m_service : null;
// Remove reference from usages array.
((ServiceRegistrationImpl.ServiceReferenceImpl) ref)