Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 10.8.1.2
-
Component/s: Network Server
-
Labels:None
Description
DRDAConnThread.closeStream():
} catch (IOException e) {
Util.javaException(e);
}
DRDAStatement.execute():
} catch (IOException e) {
Util.javaException(e);
}
Since Util.javaException() only returns an exception, this code doesn't work as intended. The returned exception should also be thrown.
} catch (IOException e) {
Util.javaException(e);
}
DRDAStatement.execute():
} catch (IOException e) {
Util.javaException(e);
}
Since Util.javaException() only returns an exception, this code doesn't work as intended. The returned exception should also be thrown.
Issue Links
- is duplicated by
-
DERBY-4350
javaException is created but not thrown or logged in DRDAConnThread.closeStream(InputStream stream)
-
- Closed
-
Activity
There is issue DERBY-4350 which I am guessing can be closed as a duplicate of this, except that that issue also mentions the lack of logging, which is not addressed with the patch, but that would still be covered by DERBY-1191.
Show
Kathey Marsden
added a comment - There is issue DERBY-4350 which I am guessing can be closed as a duplicate of this, except that that issue also mentions the lack of logging, which is not addressed with the patch, but that would still be covered by DERBY-1191 .
Thanks, Kathey, the DRDAConnThread part of this issue is the same as DERBY-4350. I'll resolve that issue.
All the regression tests passed. Committed revision 1163616.
Show
Knut Anders Hatlen
added a comment - Thanks, Kathey, the DRDAConnThread part of this issue is the same as DERBY-4350 . I'll resolve that issue.
All the regression tests passed. Committed revision 1163616.
reopen for backport.
Show
Myrna van Lunteren
added a comment - reopen for backport.
backported (with some minor manual editing of DRDAConnThread) with revision 1164755.
Show
Myrna van Lunteren
added a comment - backported (with some minor manual editing of DRDAConnThread) with revision 1164755.
The attached patch makes the suggested changes. Running regression tests now.