[jira] Closed: (FELIX-335) Configuration Admin updates with empty properties and throws NullPointerExceptions due to race condition

["Karl Pauls (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FELIX-335?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Karl Pauls closed FELIX-335.
----------------------------


I asked Ronald Spierenburg to run it through our (even more) involved tests and he wasn't able to produce any problems -- hence, I'm going to close this issue. Thanks a lot for fixing these issues so quickly!

> Configuration Admin updates with empty properties and throws NullPointerExceptions due to race condition
> --------------------------------------------------------------------------------------------------------
>
>                 Key: FELIX-335
>                 URL: https://issues.apache.org/jira/browse/FELIX-335
>             Project: Felix
>          Issue Type: Bug
>          Components: Configuration Admin
>            Reporter: Karl Pauls
>            Assignee: Felix Meschberger
>             Fix For: 1.0.0
>
>         Attachments: ConfigAdmin.FELIX-335.patch, ConfigAdmin.FELIX-335_2.patch, empty-properties.patch
>
>
> There appears to be a race condition in the configuration admin that allows managed services to get updated with empty properties (but not null). Furthermore, due to the same bug NullPointerExceptions appear under heavy load. 
> Fortunately, I think I was able to isolate the bug. The issue is that configurations are persisted when created but before they are initialized (i.e., given a configuration). If during this window a new ManagedService appears it gets updated with an empty Dictionary. Likewise, if the configuration is re-read after such an update attempt it does not have a pid which causes the null pointer exceptions.
> The fix seems to be rather simple  (unless I'm missing something :-) but took me some time to figure out. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.