AfterShip / clickhouse-sql-parser

52%

master: 52%

LAST BUILD ON BRANCH fix/nil-deref-error-formatting

branch: fix/nil-deref-error-formatting

CHANGE BRANCH

x

Reset

Sync Branches

• fix/nil-deref-error-formatting
• 14-2
• add-benchmarks
• add-coverall
• add-modify-ttl
• add-more-badges
• add-valid-until-clause
• ajruckman_alter_table_delete_where
• ajruckman_placeholder_and_select_fix
• alert-autofix-1
• allow-string-as-tablename
• allow-var-start-with-dollar
• alter-role-settings
• alter-table-drop-detached
• case-statement-when-spacing
• change-sql-string-to-compact
• claude/frosty-wiles-784a80
• claude/issue-184-20250827-0341
• claude/romantic-kowalevski-5a7ec1
• codec-delta-level
• copilot/add-beautify-format-alter-role
• copilot/add-beautify-format-for-create-user
• copilot/add-beautify-format-instruction
• copilot/add-beautify-format-support
• copilot/add-ident-break-for-insert
• copilot/add-ident-break-for-settings
• copilot/add-ident-break-from-expression
• copilot/add-ident-break-on-cluster
• copilot/add-support-in-partition-expression
• copilot/fix-12
• copilot/fix-140
• copilot/fix-179
• copilot/fix-181
• copilot/fix-187
• copilot/fix-75186bce-1dd8-4e67-bcd7-685469aa2574
• copilot/fix-e5e3e050-4a0c-4226-8f60-806cafedacba
• copilot/fix-stderr-routing-issue
• copilot/refactor-formatter-chaining-methods
• copilot/update-tool-version-to-0417
• dev
• dev1
• distinct-on
• feat/add-order-by-with-fill-support
• feat/alter_modify_query_support
• feat_support_db_comment
• feature/access-tuple-with-index
• feature/add-projection
• feature/add-union-distinct
• feature/adjust_engineExpr_String_printing_order
• feature/allow-select-modifier
• feature/alter-table-replace
• feature/check-expression
• feature/clear-projection-statement
• feature/column-def-alias-materialized
• feature/create-function
• feature/create-or-replace
• feature/create-role
• feature/delete-from
• feature/drop-projection
• feature/drop-user-role
• feature/grant-privilege
• feature/grouping-sets
• feature/insert-expr
• feature/insert-with-type-column
• feature/materialize-index-and-projection
• feature/optimize-expression
• feature/query-parameter
• feature/rename-table
• feature/select-in-function
• feature/support-json-type-skip
• feature/support-placeholder
• feature/support-ttl-policy
• feature/support-tuple
• feature/timestamp-interval
• fix-array-join-take-2
• fix-bugs-1
• fix-cannot-parse-join-only
• fix-database-column-name
• fix-describe-table
• fix-materialized-bug
• fix-no-as-alias
• fix-order-by-panic
• fix-panic-in-eof-error
• fix-select-limit-add-select-final
• fix/allow-parameters-for-null-engine
• fix/ast-walker
• fix/between-expr
• fix/between-visitor
• fix/cast-expr
• fix/column-alias-string
• fix/distinct-keyword-preservation
• fix/distinct-on-dot-notation
• fix/dollar-sign
• fix/dont_stop_walk_nil
• fix/double-quote-as-ident
• fix/group-by-column-with-keyworld
• fix/insert-into
• fix/issue-118
• fix/issue-176
• fix/issue-187
• fix/issue-222
• fix/issues-108
• fix/keyword-as-column-in-select
• fix/keyword_alias
• fix/lexer-unterminated-comment-hang
• fix/modify-ttl
• fix/multi-join
• fix/multi-line-comments
• fix/multi_distinct_except
• fix/npe-unexpected-token
• fix/panic-invalid-sql
• fix/parse-comment-literal
• fix/readme
• fix/remove-extra-leading-space
• fix/select-as-query
• fix/select-issues
• fix/string-as-table-ident
• fix/table-visitor
• fix/union-all
• fix/walk_when_node
• fix/when_case_then
• fix/window_clause
• fix/wraperror-index-out-of-range
• fix/wrong-unary-expr-string
• ft-enhance-json-options
• improve-lexer
• improve-rename-syntax
• improve/parse-expr
• issue-204
• issue-23
• json-type-hints
• lexer-string-escapes
• master
• minor-refactor-column-type
• project-group-or-order
• refactor/allow-to-use-precedence
• refactor/expect-token-kind
• refactor/operation-name
• replicated-database-fix
• role/alter
• role/on-cluster-expr
• select-format
• select-with-left-join
• settings-bool-parser
• sharad/add-comment-support-to-create-view
• sharad/bug_fix
• sharad/codec_parser
• sharad/codec_parser_v3
• sharad/delta_number_literal
• sharad/dictionary_v1
• sharad/fix-mv-comment-before-as
• sharad/fix-rmv-column-list-engine
• sharad/mv_definer
• sharad/named_parameter_v1
• sharad/table_comment
• support-concat-operator
• support-multiple-array-joins
• support-nested-tuple-definitions
• support-offset-without-limit
• support-refresh-in-mv
• support/table-alias-without-as
• swgillespie/alter-table-segv
• swgillespie/clickhouse-remotesecure
• test-automation
• unify-sql-terms
• update-readme

Build # 27258265881

Build Type

Pull #273

github

Committed by git-hulk

Commit Message

Fix nil-pointer dereference when formatting errors at EOF

Error messages built with `fmt.Errorf("... %q", p.last().String)` panic when
the lexer is at EOF, because `p.last()` is nil. Inputs like `ALTER `,
`SELECT*FROM A(0A` and `SET A=` triggered this.

Add a nil-safe `lastTokenString()` helper (mirroring `lastTokenKind()`) that
returns "<EOF>" when there is no current token, and use it in the 11 error
paths in parser_table.go and parser_alter.go. Success paths where last() is
provably non-nil are left untouched.

Add the inputs above to TestParser_InvalidSyntax to reproduce: they panicked
before the fix and now return a proper error.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Pull Request Pull Request #273: Fix nil-pointer dereference when formatting errors at EOF

Coverage Stats

9 of 16 new or added lines in 3 files covered. (56.25%)

8291 of 16009 relevant lines covered (51.79%)

3014.6 hits per line