VDB
KO
LOW

GHSA-995v-fvrw-c78m

opentelemetry-go's Schema ParseFile leaks file descriptors on each parse

Details

### Summary

`go.opentelemetry.io/otel/schema/v1.0` and `go.opentelemetry.io/otel/schema/v1.1` leaks one file descriptor on each successful `ParseFile` call. `ParseFile` opens the schema file and passes it to `Parse` without closing it; repeated parsing in a long-running process can exhaust the process file descriptor limit and cause denial of service. The severity is low because exploitation depends on a consuming application exposing repeated schema parsing to an attacker-controlled path.

Introduced in commit: e72a235

### Details

In `schema/v1.0/parser.go:41-47`, `ParseFile` opens the requested schema path with `os.Open` and then returns `Parse(file)` without a `defer file.Close()` or other close path:

```go file, err := os.Open(schemaFilePath) if err != nil { return nil, err } return Parse(file) ```

The validation evidence also identifies `schema/v1.0/parser.go:50-73`: `Parse` accepts an `io.Reader`, decodes from it, and does not close it. Ownership of the opened file is therefore not transferred to `Parse`, leaving the descriptor open until the Go runtime eventually finalizes the file object. With repeated `ParseFile` calls, descriptors can accumulate until the process receives `EMFILE` / "too many open files".

### PoC

[validation-artifact.zip](https://github.com/user-attachments/files/27494463/validation-artifact.zip)

The local artifact `validation-artifact.zip` contains:

- `leak_poc.go`: PoC source that repeatedly calls `schema.ParseFile("schema/v1.0/testdata/valid-example.yaml")` and prints `/proc/self/fd` counts. - `LEAK_POC_README.txt`: reproduction notes. - `leak_poc_run.log`: captured attempted run; the local offline environment failed before execution because Go module download from `proxy.golang.org` was forbidden.

Reproduce from the root of a checkout of `pellared/opentelemetry-go` at commit `e72a235` with Go module dependencies already available:

```sh /bin/sh -c 'ulimit -n 256; GOGC=off go run leak_poc.go' ```

Configuration:

- File descriptor soft limit: `256` - Garbage collection: disabled with `GOGC=off` so leaked descriptors are not reclaimed during the loop - Schema file: `schema/v1.0/testdata/valid-example.yaml`

Expected output is increasing descriptor counts followed by an `EMFILE` failure, for example:

```text iter 0 fds 7 iter 50 fds 57 iter 100 fds 107 ... panic: iteration 248: open schema/v1.0/testdata/valid-example.yaml: too many open files ```

The exact initial descriptor count and failing iteration can vary by OS and process state.

### Impact

This is a file descriptor resource leak leading to availability loss. Applications that call `schema.ParseFile` repeatedly, especially through a runtime reload or request-controlled path, can exhaust their process file descriptor table and fail subsequent file, socket, or other descriptor operations. Impact is limited to denial of service of the consuming process; the evidence does not show confidentiality or integrity impact.

Are you affected?

Enter the version of the package you're using.

Affected packages

Go / go.opentelemetry.io/otel/schema/v1.1
Introduced in: 0 Fixed in: 0.0.17
Fix go get go.opentelemetry.io/otel/schema/v1.1@v0.0.17
Go / go.opentelemetry.io/otel/schema/v1.0
Introduced in: 0 Fixed in: 0.0.17
Fix go get go.opentelemetry.io/otel/schema/v1.0@v0.0.17

References