You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Sunitha Kambhampati (JIRA)" <de...@db.apache.org> on 2006/07/24 18:51:14 UTC
[jira] Closed: (DERBY-1227) Network Server should not use the
underlying embedded prepared statement when accessing a
BrokeredPreparedStatement
[ http://issues.apache.org/jira/browse/DERBY-1227?page=all ]
Sunitha Kambhampati closed DERBY-1227.
--------------------------------------
Ran tests and verified the fix is in 10.1 as well as 10.2, so closing this issue.
> 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