libyaml issues

%TAG prefix does not accept all characters in ns-uri-char production

2

As noted in https://github.com/yaml/yaml-spec/issues/268#issuecomment-1208565027, Psych does not accept a `%TAG` prefix including a `#`, which seems to be due to the following code: https://github.com/yaml/libyaml/blob/f8f760f7387d2cc56a2fc7b1be313a3bf3f7f58c/src/scanner.c#L2603-L2627 According to the[YAML 1.2 Spec](https://yaml.org/spec/1.2.2/#rule-ns-uri-char) the...

gkellogg

Free version_directive if not stored for later use

4

This was an error returned from Coverity static analysis tool: Error: RESOURCE_LEAK (CWE-772): yaml-0.1.7/src/parser.c:1273: alloc_fn: Storage is returned from allocation function "yaml_malloc". yaml-0.1.7/src/api.c:33:5: alloc_fn: Storage is returned from allocation function...

hhorak

Replace the only two occurrences of \t in C code

1

The use of hard tabs, where everything else in this project uses 4-space indent, causes the code to be displayed in a misleading way when tab size is anything other...

dtolnay

Loader cycles on large input

1

How to reproduce: ```sh $ cmake . $ make $ yes '{' | head -n 1048576 | tr -d '\n' > x.yaml $ ./run-loader x.yaml ``` I don't know, but...

Totktonada

Avoid a crash when using `musl`

1

Without this PR, a warning is emitted when compiling libyaml with musl libc: ``` api.c: In function 'yaml_strdup': api.c:66:27: warning: implicit declaration of function 'strdup'; did you mean 'strcmp'? [-Wimplicit-function-declaration]...

matttbe

Support Unicode Surrogate Pairs

1

While they are not explicitly mentioned in the YAML 1.1 or 1.2 spec, the following is valid JSON: ``` { "unicode": "surrogate pair \ud83d\udcaf" } ``` Same as ``` {...

perlpunk