Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

yacc issue: index out of range #1801

Closed
tbg opened this issue Jul 24, 2015 · 1 comment
Closed

yacc issue: index out of range #1801

tbg opened this issue Jul 24, 2015 · 1 comment
Assignees
@petermattis

Comments

@tbg
Copy link
Member

tbg commented Jul 24, 2015

likely identical twin of vitessio/vitess#767

    s := "SELECT(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(T" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F(F" +
        "(F(F(F(F(F(F(F(F(F((" +
        "F(0"

    Parse(s)

test code in #1802

stack trace:

goroutine 1 [running]:
github.com/cockroachdb/cockroach/sql/parser.sqlParse(0x2208386ac0, 0x2083ba400, 0x0)
/var/folders/yy/4q8rrssd27vdgbr59w9qbffr0000gn/T/go-fuzz-build896428593/src/github.com/cockroachdb/cockroach/sql/parser/yaccpar:207 +0x548df
github.com/cockroachdb/cockroach/sql/parser.Parse(0x2083b1380, 0xcb, 0x0, 0x0, 0x0, 0x0, 0x0)
/var/folders/yy/4q8rrssd27vdgbr59w9qbffr0000gn/T/go-fuzz-build896428593/src/github.com/cockroachdb/cockroach/sql/parser/parse.go:50 +0x16a
tbg added a commit to tbg/cockroach that referenced this issue Jul 24, 2015
@tbg
add a test documenting cockroachdb#1801
1c3aa5a 
it would currently fail, so it is skipped.
tbg added a commit that referenced this issue Jul 24, 2015
@tbg
Merge pull request #1802 from tschottdorf/yacc_panic
8b4e232 
add a test documenting #1801
@petermattis
Copy link
Collaborator

This seems to be some problem in the generated code during error recovery when sqlMaxDepth (default 200) is reached.

@petermattis petermattis mentioned this issue Jul 24, 2015
Merged
@petermattis petermattis mentioned this issue Jul 24, 2015
Merged
@petermattis petermattis self-assigned this Jul 24, 2015
petermattis added a commit that referenced this issue Jul 24, 2015
@petermattis
Bump the yacc pointer.
8bfe748 
Use the newer yacc to re-generate the sql grammar.

Fixes #1801.
@petermattis petermattis removed the PTAL label Jul 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@petermattis petermattis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tbg @petermattis