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

revved gopkg.in/yaml.v3 to v3.0.1 to fix CVE-2022-28948 #1077

Closed
wants to merge 2 commits into from

Conversation

kylefossum
Copy link

Hello,

submitting this PR to fix a vulnerability found by a container scan.

https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-28948

{
          "hash": "",
          "created_by": "dpkg -i sops_3.7.3_amd64.deb",
          "packages": [
            {
              "name": "gopkg.in/yaml.v3",
              "namespace": "go",
              "version": "v3.0.0-20210107192922-496545a6307b",
              "src": "usr/local/bin/sops",
              "vulnerabilities": [
                {
                  "name": "CVE-2022-28948",
                  "description": "An issue in the Unmarshal function in Go-Yaml v3 causes the program to crash when attempting to deserialize invalid input.",
                  "severity": "High",
                  "link": "https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-28948",
                  "fix_version": "3.0.0",
                  "metadata": {
                    "NVD": {
                      "CVSSv2": {
                        "PublishedDateTime": "2022-05-19T20:15Z",
                        "Score": 5,
                        "Vectors": "AV:N/AC:L/Au:N/C:N/I:N/A:P"
                      },
                      "CVSSv3": {
                        "ExploitabilityScore": 3.9,
                        "ImpactScore": 3.6,
                        "Score": 7.5,
                        "Vectors": "CVSS:3.0/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"
                      }
                    }
                  },
                  "status": "VULNERABLE"
                }
              ]
            }
          ]
        },

@ajvb ajvb self-requested a review June 27, 2022 18:22
Copy link
Contributor

@ajvb ajvb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go.sum has not been updated

@kylefossum
Copy link
Author

@ajvb updated the dependency in question using the package manager. I think that's what we needed... first time contributing to a go project so thanks for bearing with me.

@hiddeco
Copy link
Member

hiddeco commented Jul 3, 2023

As #1147 will deal with this as well (and a literal update of all other outdated things). I am going to favor that PR over this one. However, I do want to thank you for your contribution! 🌷

@hiddeco hiddeco closed this Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants