cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 30874] - SQLTransformer throws exception with stored procs on Sybase
Date Thu, 26 Aug 2004 22:04:29 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=30874>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30874

SQLTransformer throws exception with stored procs on Sybase





------- Additional Comments From ats37@hotmail.com  2004-08-26 22:04 -------
The obvious (and trivial) fix, of course, is to replace

                if ( pst != null )
                    pst.close();
                pst = null;        // Prevent using pst again.
                if ( cst != null )
                    cst.close();
                cst = null;        // Prevent using cst again.

with

                if ( pst != null && pst != cst )
                    pst.close();
                pst = null;        // Prevent using pst again.
                if ( cst != null )
                    cst.close();
                cst = null;        // Prevent using cst again.

without any risk of breaking anything, if for some reason both of them are ever
(legitimately) used and different (which I doubt).

I'd offer to upload a patch, but for 14 characters it hardly seems worth it ;-)

Mime
View raw message