[JCR-1246] In DatabasePersistenceManager.store(), if the exception is null or its cause is not an SQLException, then the PM keeps looping forever

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3, 1.3.1, 1.3.3
• Fix Version/s: 1.4
• Component/s: jackrabbit-core
• Environment: Windows XP x64
• Assignee: Stefan Guggisberg
• Reporter: Esteban Franqueiro
• Created: 29/Nov/07 20:19
• Updated: 15/Jan/08 23:27
• Resolved: 30/Nov/07 10:34

Description

In the line
if (ise != null && ise.getCause() instanceof SQLException && --trials > 0) {
if one of the first two checks fails, the shortcircuit doesn't decrement trials.