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.

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

Memory leak with trailing quotes #138

Open
MichaelHatherly opened this issue Apr 17, 2024 · 2 comments
Open

Memory leak with trailing quotes #138

MichaelHatherly opened this issue Apr 17, 2024 · 2 comments

Comments

@MichaelHatherly
Copy link

The following code hangs and continues to consume more memory until you manually kill it.

YAML.load("a: \"b\"\"")

Note the trailing " after "b".

image

with package versions on Julia 1.10.2

(@v1.10) pkg> activate --temp
  Activating new project at `/var/folders/09/t9pc5r114yg9pfrf28lb2d440000gn/T/jl_DegVYD`

(jl_DegVYD) pkg> add YAML
   Resolving package versions...
    Updating `/private/var/folders/09/t9pc5r114yg9pfrf28lb2d440000gn/T/jl_DegVYD/Project.toml`
  [ddb6d928] + YAML v0.4.9
    Updating `/private/var/folders/09/t9pc5r114yg9pfrf28lb2d440000gn/T/jl_DegVYD/Manifest.toml`
  [692b3bcd] + JLLWrappers v1.5.0
  [21216c6a] + Preferences v1.4.3
  [69024149] + StringEncodings v0.3.7
  [ddb6d928] + YAML v0.4.9
  [94ce4f54] + Libiconv_jll v1.17.0+0
  [56f22d72] + Artifacts
  [2a0f44e3] + Base64
  [ade2ca70] + Dates
  [8f399da3] + Libdl
  [de0858da] + Printf
  [fa267f1f] + TOML v1.0.3
  [4ec0a83e] + Unicode
@kescobo kescobo added the bug label Apr 18, 2024
@kescobo
Copy link
Collaborator

kescobo commented Apr 19, 2024

That seems bad 🤔

It should probably be a syntax error, right?

image

@MichaelHatherly
Copy link
Author

It should probably be a syntax error, right?

Yeah, sounds right to me.

@kescobo kescobo added the parser label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants