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

x/vulndb: potential Go vuln in github.com/gagliardetto/binary: CVE-2022-36078 #963

Closed
GoVulnBot opened this issue Sep 2, 2022 · 1 comment
Assignees

Comments

@GoVulnBot
Copy link

CVE-2022-36078 references github.com/gagliardetto/binary, which may be a Go module.

Description:
Binary provides encoding/decoding in Borsh and other formats. The vulnerability is a memory allocation vulnerability that can be exploited to allocate slices in memory with (arbitrary) excessive size value, which can either exhaust available memory or crash the whole program. When using github.com/gagliardetto/binary to parse unchecked (or wrong type of) data from untrusted sources of input (e.g. the blockchain) into slices, it's possible to allocate memory with excessive size. When dec.Decode(&val) method is used to parse data into a structure that is or contains slices of values, the length of the slice was previously read directly from the data itself without any checks on the size of it, and then a slice was allocated. This could lead to an overflow and an allocation of memory with excessive size value. Users should upgrade to v0.7.1 or higher. A workaround is not to rely on the dec.Decode(&val) function to parse the data, but to use a custom UnmarshalWithDecoder() method that reads and checks the length of any slice.

References:

See doc/triage.md for instructions on how to triage this report.

modules:
  - module: github.com/gagliardetto/binary
    packages:
      - package: binary
description: |
    Binary provides encoding/decoding in Borsh and other formats. The vulnerability is a memory allocation vulnerability that can be exploited to allocate slices in memory with (arbitrary) excessive size value, which can either exhaust available memory or crash the whole program. When using `github.com/gagliardetto/binary` to parse unchecked (or wrong type of) data from untrusted sources of input (e.g. the blockchain) into slices, it's possible to allocate memory with excessive size. When `dec.Decode(&val)` method is used to parse data into a structure that is or contains slices of values, the length of the slice was previously read directly from the data itself without any checks on the size of it, and then a slice was allocated. This could lead to an overflow and an allocation of memory with excessive size value. Users should upgrade to `v0.7.1` or higher. A workaround is not to rely on the `dec.Decode(&val)` function to parse the data, but to use a custom `UnmarshalWithDecoder()` method that reads and checks the length of any slice.
cves:
  - CVE-2022-36078
references:
  - web: https://github.com/gagliardetto/binary/security/advisories/GHSA-4p6f-m4f9-ch88
  - fix: https://github.com/gagliardetto/binary/pull/7
  - web: https://github.com/gagliardetto/binary/releases/tag/v0.7.1

@tatianab tatianab self-assigned this Sep 2, 2022
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/427979 mentions this issue: data/reports: add GO-2022-0963.yaml for CVE-2022-36078

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

3 participants