Is this code change just masking the underlying cause of the problem?  Is there any concern with catching all Exceptions and just swallowing them?  What is the drawback if an exception is thrown and handled in this manner?  For example, uPortal will startup but some features will not work?