Issue Details (XML | Word | Printable)

Key: CORE-5382
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Dmitry Yemanov
Reporter: Kovalenko Dmitry
Votes: 1
Watchers: 2
Operations

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

Incorrect processing (truncation) of SQL statement longer than 10MB

Created: 26/Oct/16 12:26 PM   Updated: 08/Nov/16 10:25 AM
Component/s: Engine
Affects Version/s: 4.0 Initial, 3.0.0, 3.0.1
Fix Version/s: 3.0.2, 4.0 Alpha 1

Environment: Firebird 3.0.2.32619, IBProvider, direct INET-connection to FB3 (without fbclient.dll).

QA Status: Covered by another test(s)
Test Details: See fbt-repo\tests\bugs\isql_05.fbt


 Description  « Hide
SQL Statement (total length = 10*1024*1024+1):

 "select/*---------- .... ---------------*/ID from DUAL "

---------------
Prepare error:

Dynamic SQL Error
SQL error code = -204
Table unknown
DUA

---------------

1. I think, the error message must be a little another.

2. Special for this case, I created the new table "DUA" and server prepared original query without any problem.


 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Adriano dos Santos Fernandes added a comment - 03/Nov/16 03:20 PM
10 MB is the limit and the code truncates (by purpose) it.

Kovalenko Dmitry added a comment - 04/Nov/16 05:31 AM - edited
Sorry, I not understood - FB source code was corrected or no?

Dmitry Yemanov added a comment - 04/Nov/16 05:48 AM
Sorry, Pavel was too fast to resolve this ticket. The current behaviour is intended, but honestly, I don't remember why I decided to truncate instead of raising the error.

Dmitry Yemanov added a comment - 04/Nov/16 03:57 PM
Would there be any objections to replacing silent truncation with an explicit error?

Adriano dos Santos Fernandes added a comment - 04/Nov/16 04:55 PM
> Would there be any objections to replacing silent truncation with an explicit error?

Not from me, I agree it should throw.

Dmitry Yemanov added a comment - 08/Nov/16 10:25 AM
Fixed to raise an error instead of truncating.