golang / vulndb

[mirror] The Go Vulnerability Database
Other
558 stars 56 forks source link

x/vulndb: potential Go vuln in github.com/codenotary/immudb/pkg/client: GHSA-6cqj-6969-p57x #1118

Closed GoVulnBot closed 1 year ago

GoVulnBot commented 1 year ago

In GitHub Security Advisory GHSA-6cqj-6969-p57x, there is a vulnerability in the following Go packages or modules:

Unit Fixed Vulnerable Ranges
github.com/codenotary/immudb/pkg/client 1.4.1 < 1.4.1

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

modules:
  - module: TODO
    versions:
      - fixed: 1.4.1
    packages:
      - package: github.com/codenotary/immudb/pkg/client
description: |
    ### Impact

    immudb client SDKs use server's UUID to distinguish between different server instance so that the client can connect to different immudb instances and keep the state for multiple servers. SDK does not validate this uuid and can accept any value reported by the server. A malicious server can change the reported UUID tricking the client to treat it as a different server thus accepting a state completely irrelevant to the one previously retrieved from the server.

    ### Patches

    The following Go SDK versions are not vulnerable

    | **SDK** | **Version** |
    |-------|------------|
    | [go](pkg.go.dev/github.com/codenotary/immudb/pkg/client) | 1.4.1 |

    ### Workarounds

    When initializing an immudb client object, a custom state handler can be used to store the state. Providing custom implementation that ignores the server UUID can be used to ensure that even if the server changes the UUID, client will still consider it to be the same server.

    ### For more information

    If you have any questions or comments about this advisory:

    * Open a discussion in [immudb Discussions](https://github.com/codenotary/immudb/discussions/new)
    * Email us at [immudb-security@codenotary.com](mailto:immudb-security@codenotary.com)
cves:
  - CVE-2022-39199
ghsas:
  - GHSA-6cqj-6969-p57x
tatianab commented 1 year ago

fix:

possibly related:

gopherbot commented 1 year ago

Change https://go.dev/cl/459219 mentions this issue: data/reports: add GO-2022-1118.yaml