[ http://issues.apache.org/jira/browse/DERBY-1227?page=all ]
Mike Matrigali updated DERBY-1227:
----------------------------------
Fix Version/s: 10.1.4.0
committed patch to 10.1 branch:
m101_142:117>svn commit
Sending java\drda\org\apache\derby\impl\drda\DRDAConnThread.java
Sending java\drda\org\apache\derby\impl\drda\DRDAStatement.java
Sending
java\engine\org\apache\derby\iapi\jdbc\BrokeredPreparedStatement.java
Adding
java\engine\org\apache\derby\iapi\jdbc\EngineParameterMetaData.java
Adding
java\engine\org\apache\derby\iapi\jdbc\EnginePreparedStatement.java
Sending
java\engine\org\apache\derby\impl\jdbc\EmbedParameterSetMetaData.java
Sending
java\engine\org\apache\derby\impl\jdbc\EmbedPreparedStatement.java
Transmitting file data .......
Committed revision 424451.
> Network Server should not use the underlying embedded prepared statement when
> accessing a BrokeredPreparedStatement
> -------------------------------------------------------------------------------------------------------------------
>
> Key: DERBY-1227
> URL: http://issues.apache.org/jira/browse/DERBY-1227
> Project: Derby
> Issue Type: Bug
> Components: Network Server
> Affects Versions: 10.1.2.1
> Reporter: Kathey Marsden
> Assigned To: Sunitha Kambhampati
> Fix For: 10.2.0.0, 10.1.4.0
>
> Attachments: 10.1_p1_derby1227.diff.txt
>
>
> Network Server should not use the underlying embedded
> Prepared statement when accessing a BrokeredPreparedStatement.
> In DERBY-1025 Dan pointed out this code which while not related to DERBY-1025
> does appear to be a problem. Here is his comment regarding this code in
> DRDAStatement:
> /**
> * Get prepared statement
> *
> * @return prepared statement
> */
> protected PreparedStatement getPreparedStatement() throws SQLException
> {
> if (ps instanceof BrokeredPreparedStatement)
> return (PreparedStatement)(
> ((BrokeredPreparedStatement) ps).getStatement());
> else
> return ps;
> }
> This code, for some unknown reason due to lack of comments, is getting the
> underlying embedded statement
> from a BrokeredPreparedStatement. This should not be allowed, the
> BrokeredStatement wrappers are there to
> hide the embedded statement object as it can change under the covers of the
> wrapper.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
