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

Kurt T Stam closed JUDDI-375.
-----------------------------

    Resolution: Fixed

fixed

> Null Pointer Exception in SubscriptionNotifier
> ----------------------------------------------
>
>                 Key: JUDDI-375
>                 URL: https://issues.apache.org/jira/browse/JUDDI-375
>             Project: jUDDI
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 3.0.1
>         Environment: org.apache.juddi.subscription.SubscriptionNotifier
> When load becomes significant and the notifier falls behind.
>            Reporter: Adam Pryce
>            Assignee: Kurt T Stam
>            Priority: Minor
>             Fix For: 3.0.2
>
>
> When the load becomes significant it is possible for the following if 
> statement at line 144 to resolve to false resulting in a null pointer being 
> returned
> 144:          if (subscription.getLastNotified()==null || 
> nextDesiredNotificationDate.after(startPoint) && 
> nextDesiredNotificationDate.before(endPoint)) {
> If this if statement is sensible then there needs to be a null pointer check 
> around line 87 to avoid NullPointerException. I could see something like this:
> 86:   GetSubscriptionResults getSubscriptionResults = 
> buildGetSubscriptionResults(subscription, new Date(scheduledExecutionTime()));
>         if(getSubscriptionResults == null)
>             continue;              
> 87:   
> getSubscriptionResults.setSubscriptionKey(subscription.getSubscriptionKey());
> I apologize that I cannot test and commit this change through subversion. I 
> put priority to minor since the NPE seems to have relatively little impact in 
> the long term, but in the short term all remaining subscriptions are skipped.

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

Reply via email to