AfterShip / clickhouse-sql-parser / 27258626839

52%

push

github

Fix infinite loop on unterminated block comment (#271)

`consumeMultiLineComment` looped on `!l.isEOF()` (which checks `l.current`)
while only advancing the local index `i`, so on an unclosed comment like
`/* unterminated` the lexer spun forever — a DoS hang.

Loop on `l.peekOk(i)` instead so it terminates at end of input.

Add TestConsumeUnterminatedComment to reproduce: it drains the lexer in a
goroutine with a timeout, so a regression fails fast instead of hanging.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

1 of 1 new or added line in 1 file covered. (100.0%)

8291 of 16009 relevant lines covered (51.79%)

3014.67 hits per line