Issue Details (XML | Word | Printable)

Key: DERBY-488
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Kathey Marsden
Reporter: Daniel John Debrunner
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Derby

DatabaseMetaData.getColumns() fails on iSeries JDK 1.4 with verfier error on generated class.

Created: 04/Aug/05 05:24 AM   Updated: 30/Sep/05 02:27 AM
Return to search
Component/s: SQL
Affects Version/s: 10.1.1.0
Fix Version/s: 10.0.2.2, 10.1.2.1, 10.2.1.6

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works patch488.txt 2005-08-04 05:26 AM Daniel John Debrunner 0.6 kB

Resolution Date: 30/Sep/05 02:27 AM


 Description  « Hide
Analysis shows that
----------------------------------------------------------------------
The problem is occurring starting at offset 2007 in method e23. There is an invokeinterface to method setWidth(int, int, boolean) of class VariableSizeDataValue. This invoke returns a value of class DataValueDescriptor. That value is in turn stored in field e142 at offset 2015 in method e23. The problem is that field e142 is a NumberDataValue, and DataValueDescriptor is not a valid subclass of NumberDataValue. Thus the store is not allowed.
----------------------------------------------------------------------

Looking at the generated setWidth() calls I see one in BinaryOperatorNode where the return (DataValueDescriptor) is not cast to the type of the field it is stored in.


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Daniel John Debrunner added a comment - 04/Aug/05 05:26 AM
Added patch
Also meant to add that it's unclear why other verifiers on other platforms don't catch this.

Daniel John Debrunner added a comment - 04/Aug/05 11:22 PM
Committed revision (10.1 branch) 227406

Daniel John Debrunner added a comment - 05/Aug/05 08:45 AM
Fix from 10.1 merged to trunk as revision 227518

Daniel John Debrunner added a comment - 05/Aug/05 08:46 AM
Fix verified on iSeries hardware.

Deepa Remesh added a comment - 17/Aug/05 03:39 AM
This error has to be ported to 10.0 branch also.

Andrew McIntyre added a comment - 20/Aug/05 05:47 AM
changing fixin to 10.1.1.1

Kathey Marsden added a comment - 20/Aug/05 05:51 AM
ported to 10.0 branch

Deepa Remesh added a comment - 30/Sep/05 02:27 AM
reopening to fix fix version