Software Updates enters feedback loop if pre-exposed to an error

[Ken McLean]

My instance of OpenScore encountered a bug when operating (namely bug #310419). However this error only displays when using Debug mode, and a regular user would not be aware any error.

I then tried to use the Check for Updates > Software Updates function, which encountered a feedback loop, spitting out errors and creating a series of new log files. The application closed (as per bug #311522), and gave the following errors in log:

22:34:40:INFO :Main Jar Dir: file:/C:/Program%20Files/OpenScore/
22:34:40:ERROR:Unhandled exception:

[new error log file]

[2008-12-26]
22:34:41:INFO :2:40=>2:40.00[OK]
22:34:41:INFO :DNF=>DNF[OK]
22:34:41:INFO :DNS=>DNS[OK]
22:34:41:INFO :2:40DQ=>2:40.00DQ[OK]
22:34:41:INFO :12345=>1:23:45.00[OK]
22:34:41:INFO :12345dq=>1:23:45.00DQ[OK]
22:34:41:INFO :123.45=>1:23.45[OK]
22:34:41:INFO :MCKINNON=>McKinnon[OK]
22:34:41:INFO :MACUSER MCTEST O'CONNOR=>MacUser McTest O'Connor[OK]
22:34:41:INFO :A B C=>A B C[OK]
22:34:41:ERROR:Base error caught:java.lang.NullPointerException
 at avscoreswt.gui.BaseFrame.updateMeetMenu(BaseFrame.java:374)
 at avscoreswt.gui.BaseFrame.triggerMenus(BaseFrame.java:302)
 at avscoreswt.gui.BaseFrame.buildUI(BaseFrame.java:832)
 at avscoreswt.gui.BaseFrame.<init>(BaseFrame.java:129)
 at avscoreswt.gui.BaseFrame.start(BaseFrame.java:842)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
 at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
 at java.lang.reflect.Method.invoke(Unknown Source)
 at avscoreswt.OpenScore.main(OpenScore.java:41)

[2008-12-26]
22:34:41:INFO :2:40=>2:40.00[OK]
22:34:41:INFO :DNF=>DNF[OK]
22:34:41:INFO :DNS=>DNS[OK]
22:34:41:INFO :2:40DQ=>2:40.00DQ[OK]
22:34:41:INFO :12345=>1:23:45.00[OK]
22:34:41:INFO :12345dq=>1:23:45.00DQ[OK]
22:34:41:INFO :123.45=>1:23.45[OK]
22:34:41:INFO :MCKINNON=>McKinnon[OK]
22:34:41:INFO :MACUSER MCTEST O'CONNOR=>MacUser McTest O'Connor[OK]
22:34:41:INFO :A B C=>A B C[OK]
22:34:41:ERROR:Base error caught:java.lang.NullPointerException
 at avscoreswt.gui.BaseFrame.updateMeetMenu(BaseFrame.java:374)
 at avscoreswt.gui.BaseFrame.triggerMenus(BaseFrame.java:302)
 at avscoreswt.gui.BaseFrame.buildUI(BaseFrame.java:832)
 at avscoreswt.gui.BaseFrame.<init>(BaseFrame.java:129)
 at avscoreswt.gui.BaseFrame.start(BaseFrame.java:842)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
 at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
 at java.lang.reflect.Method.invoke(Unknown Source)
 at avscoreswt.OpenScore.main(OpenScore.java:41)

and then it spat out the error:

[new error log file]

[2008-12-26]
22:36:10:INFO :2:40=>2:40.00[OK]
22:36:10:INFO :DNF=>DNF[OK]
22:36:10:INFO :DNS=>DNS[OK]
22:36:10:INFO :2:40DQ=>2:40.00DQ[OK]
22:36:10:INFO :12345=>1:23:45.00[OK]
22:36:10:INFO :12345dq=>1:23:45.00DQ[OK]
22:36:10:INFO :123.45=>1:23.45[OK]
22:36:10:INFO :MCKINNON=>McKinnon[OK]
22:36:10:INFO :MACUSER MCTEST O'CONNOR=>MacUser McTest O'Connor[OK]
22:36:10:INFO :A B C=>A B C[OK]
22:36:10:ERROR:Base error caught:java.lang.NullPointerException
 at avscoreswt.gui.BaseFrame.updateMeetMenu(BaseFrame.java:374)
 at avscoreswt.gui.BaseFrame.triggerMenus(BaseFrame.java:302)
 at avscoreswt.gui.BaseFrame.buildUI(BaseFrame.java:832)
 at avscoreswt.gui.BaseFrame.<init>(BaseFrame.java:129)
 at avscoreswt.gui.BaseFrame.start(BaseFrame.java:842)
 at sun.reflect.GeneratedMethodAccessor1.invoke(Unknown Source)
 at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
 at java.lang.reflect.Method.invoke(Unknown Source)
 at avscoreswt.OpenScore.main(OpenScore.java:41)

[new error log file]

Repeated until the application was closed (by closing the debug command window)

I am reporting this as a bug, because a user in normal operation mode would have just experienced an application crash, and would not be aware of the endless loop cycle being performed as a process. The Update needs to try and drop all the preloaded instances of itself so that a hidden bug doesn't cause a memory leak that a user cannot stop.