[cvs] expresso commit by lhamel: protect against NPE

JCorporate Ltd jcorp at jcorporate.com
Sat Mar 5 02:02:33 UTC 2005


Log Message:
-----------
protect against NPE

Modified Files:
--------------
    expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab:
        Crontab.java

Revision Data
-------------
Index: Crontab.java
===================================================================
RCS file: /home/javacorp/.cvs/expresso/expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab/Crontab.java,v
retrieving revision 1.13
retrieving revision 1.14
diff -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab/Crontab.java -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab/Crontab.java -u -r1.13 -r1.14
--- expresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab/Crontab.java
+++ expresso-web/WEB-INF/src/com/jcorporate/expresso/services/crontab/Crontab.java
@@ -268,7 +268,14 @@
         }
 
         queue.add(entry);
-        waiter.update(((CrontabEntry) queue.first()).alarmTime);
+
+        // protect against nulls; was getting null exceptions
+        if ( waiter != null ) {
+            CrontabEntry anEntry = (CrontabEntry) queue.first();
+            if ( anEntry != null ) {
+                waiter.update(anEntry.alarmTime);
+            }
+        }
     }
 
     /**


More information about the cvs mailing list