Bug 1065854 - PreparedStatement and ResultSet objects not closed correctly from SQL JSTL tags
Summary: PreparedStatement and ResultSet objects not closed correctly from SQL JSTL tags
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: unspecified
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: DR1
: EAP 6.3.0
Assignee: Shelly McGowan
QA Contact: Jan Stefl
URL:
Whiteboard:
Depends On: 1067151 1079497
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-17 06:50 UTC by James Livingston
Modified: 2019-08-19 12:42 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-08-19 12:42:59 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Apache Bugzilla 17388 0 None None None Never
Red Hat Issue Tracker JBEE-147 0 Major Resolved QueryTagSupport does not close prepared statements and result sets correctly 2018-02-09 09:29:02 UTC

Description James Livingston 2014-02-17 06:50:40 UTC 
When using the JSTL SQL tags, PreparedStatement and ResultSet objects are not closed correct when some errors occur.

This was fixed in the upstream Apache Taglib code (https://issues.apache.org/bugzilla/show_bug.cgi?id=17388) but the fix was not backported to our fork.
Comment 1 JBoss JIRA Server 2014-02-19 19:23:47 UTC 
Shelly McGowan <smcgowan> updated the status of jira JBEE-147 to Resolved
Comment 3 Shelly McGowan 2014-02-19 20:06:35 UTC 
Note this change was reviewed by rmaucher, passes the JSTL TCK tests and the EE 6 signature tests.
Comment 5 Martin Velas 2014-09-18 14:38:32 UTC 
I have verified  that the patch http://svn.apache.org/viewvc/jakarta/taglibs/proper/standard/trunk/src/org/apache/taglibs/standard/tag/common/sql/QueryTagSupport.java?view=diff&r1=499149&r2=499150&pathrev=499150 was used in jboss-jstl-api_1.2_spec for both the version 1.0.5.Final and 1.0.6.Final (used in EAP 6.3.0.GA).

This change was also verified by JSTL TCK tests and the EE 6 signature tests - see above.
Note You need to log in before you can comment on or make changes to this bug.