diff --git a/advisories/github-reviewed/2025/01/GHSA-6wxm-mpqj-6jpf/GHSA-6wxm-mpqj-6jpf.json b/advisories/github-reviewed/2025/01/GHSA-6wxm-mpqj-6jpf/GHSA-6wxm-mpqj-6jpf.json new file mode 100644 index 00000000000..f586c08b978 --- /dev/null +++ b/advisories/github-reviewed/2025/01/GHSA-6wxm-mpqj-6jpf/GHSA-6wxm-mpqj-6jpf.json @@ -0,0 +1,78 @@ +{ + "schema_version": "1.4.0", + "id": "GHSA-6wxm-mpqj-6jpf", + "modified": "2025-01-28T17:29:03Z", + "published": "2025-01-28T17:29:03Z", + "aliases": [ + "CVE-2024-45339" + ], + "summary": "Insecure Temporary File usage in github.com/golang/glog", + "details": "When logs are written to a widely-writable directory (the default), an unprivileged attacker may predict a privileged process's log file path and pre-create a symbolic link to a sensitive file in its place. When that privileged process runs, it will follow the planted symlink and overwrite that sensitive file. To fix that, glog now causes the program to exit (with status code 2) when it finds that the configured log file already exists.", + "severity": [ + { + "type": "CVSS_V4", + "score": "CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:N/VI:H/VA:N/SC:N/SI:N/SA:N/E:U" + } + ], + "affected": [ + { + "package": { + "ecosystem": "Go", + "name": "github.com/golang/glog" + }, + "ranges": [ + { + "type": "ECOSYSTEM", + "events": [ + { + "introduced": "0" + }, + { + "fixed": "1.2.4" + } + ] + } + ] + } + ], + "references": [ + { + "type": "ADVISORY", + "url": "https://nvd.nist.gov/vuln/detail/CVE-2024-45339" + }, + { + "type": "WEB", + "url": "https://github.com/golang/glog/pull/74" + }, + { + "type": "WEB", + "url": "https://github.com/golang/glog/pull/74/commits/b8741656e406e66d6992bc2c9575e460ecaa0ec2" + }, + { + "type": "PACKAGE", + "url": "https://github.com/golang/glog" + }, + { + "type": "WEB", + "url": "https://groups.google.com/g/golang-announce/c/H-Q4ouHWyKs" + }, + { + "type": "WEB", + "url": "https://owasp.org/www-community/vulnerabilities/Insecure_Temporary_File" + }, + { + "type": "WEB", + "url": "https://pkg.go.dev/vuln/GO-2025-3372" + } + ], + "database_specific": { + "cwe_ids": [ + "CWE-377", + "CWE-61" + ], + "severity": "MODERATE", + "github_reviewed": true, + "github_reviewed_at": "2025-01-28T17:29:03Z", + "nvd_published_at": "2025-01-28T02:15:28Z" + } +} \ No newline at end of file diff --git a/advisories/github-reviewed/2025/01/GHSA-wwx5-gpgr-vxr7/GHSA-wwx5-gpgr-vxr7.json b/advisories/github-reviewed/2025/01/GHSA-wwx5-gpgr-vxr7/GHSA-wwx5-gpgr-vxr7.json new file mode 100644 index 00000000000..f0f1f65710e --- /dev/null +++ b/advisories/github-reviewed/2025/01/GHSA-wwx5-gpgr-vxr7/GHSA-wwx5-gpgr-vxr7.json @@ -0,0 +1,111 @@ +{ + "schema_version": "1.4.0", + "id": "GHSA-wwx5-gpgr-vxr7", + "modified": "2025-01-28T17:29:17Z", + "published": "2025-01-28T17:29:17Z", + "aliases": [ + "CVE-2025-24800" + ], + "summary": "ismp-grandpa crate accepted incorrect signatures", + "details": "A critical vulnerability was discovered in the `ismp-grandpa` crate, that allowed a malicious prover easily convince the verifier of the finality of arbitrary headers.\n\n### Description\n\nThe vulnerability manifests as a verifer that only accepts incorrect signatures of Grandpa precommits and was introduced in this [specific commit](https://github.com/polytope-labs/ismp-substrate/pull/64/commits/5ca3351a19151f1a439c30d5cbdbfdc72a11f1a8#diff-3835cc24fb2011b3e8246036059acd8c2c2a9a869eedf7a210d18edb6543318dL262). Perhaps due to unfamiliarity with core substrate APIs. The `if` statement should have included a negation check, similar to the previous code, but this was omitted. Causing the verifier to **only** accept invalid signatures.\n\nThis vulnerability remained undetected even with [integration tests](https://github.com/polytope-labs/ismp-substrate/pull/64/commits/04d5be207b082eb61d586d52e1685e2e060347e6#diff-4aedbca82d26bebc03f274e23fd5697c3346ffff54405c87af9018f3aef708b2R1-R160), as the prover was also [misconfigured](https://github.com/polytope-labs/ismp-substrate/pull/64/commits/b26894913b301061b07db61af841ca2586415f08#diff-493a6129d75fe31185e28695a4d2adc1582fe9df12462e380fe994f170fc1e70L159) to initialize the Grandpa verifier with the incorrect authority `set_id`. This causes verification of honest precommit signatures to fail as the message is now malformed, but the verifier indeed only accepts signatures or messages that fail the verification check.\n\nBut even more devastatingly, the verifier will also accept malicious GRANDPA signatures for any precommit message.\n\nThis vulnerability has been fixed in this [commit](https://github.com/polytope-labs/hyperbridge/pull/372/commits/f0e85db718f5165b06585a49b14a66f8ad643aea) and a patch release has been published.\n\n### Impact\nThis could be used to steal funds or compromise other kinds of cross-chain applications.\n\n### Patches\nThis vulnerability has been fixed in the latest version of `ismp-granpda` `v15.0.1`\n\n### Recommendations\nUsers who rely on the compromised versions must upgrade immediately, as all vulnerable versions of the crate has been yanked.\n", + "severity": [ + { + "type": "CVSS_V4", + "score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:H/VI:H/VA:N/SC:N/SI:N/SA:N" + } + ], + "affected": [ + { + "package": { + "ecosystem": "crates.io", + "name": "ismp-grandpa" + }, + "ranges": [ + { + "type": "ECOSYSTEM", + "events": [ + { + "introduced": "0" + }, + { + "fixed": "15.0.1" + } + ] + } + ] + }, + { + "package": { + "ecosystem": "crates.io", + "name": "grandpa-verifier-primitives" + }, + "ranges": [ + { + "type": "ECOSYSTEM", + "events": [ + { + "introduced": "0" + }, + { + "fixed": "0.1.2" + } + ] + } + ] + }, + { + "package": { + "ecosystem": "crates.io", + "name": "grandpa-verifier" + }, + "ranges": [ + { + "type": "ECOSYSTEM", + "events": [ + { + "introduced": "0" + }, + { + "fixed": "0.1.2" + } + ] + } + ] + } + ], + "references": [ + { + "type": "WEB", + "url": "https://github.com/polytope-labs/hyperbridge/security/advisories/GHSA-wwx5-gpgr-vxr7" + }, + { + "type": "WEB", + "url": "https://github.com/polytope-labs/hyperbridge/pull/372/commits/f0e85db718f5165b06585a49b14a66f8ad643aea" + }, + { + "type": "WEB", + "url": "https://github.com/polytope-labs/ismp-substrate/pull/64/commits/04d5be207b082eb61d586d52e1685e2e060347e6#diff-4aedbca82d26bebc03f274e23fd5697c3346ffff54405c87af9018f3aef708b2R1-R160" + }, + { + "type": "WEB", + "url": "https://github.com/polytope-labs/ismp-substrate/pull/64/commits/5ca3351a19151f1a439c30d5cbdbfdc72a11f1a8#diff-3835cc24fb2011b3e8246036059acd8c2c2a9a869eedf7a210d18edb6543318dL262" + }, + { + "type": "WEB", + "url": "https://github.com/polytope-labs/ismp-substrate/pull/64/commits/b26894913b301061b07db61af841ca2586415f08#diff-493a6129d75fe31185e28695a4d2adc1582fe9df12462e380fe994f170fc1e70L159" + }, + { + "type": "PACKAGE", + "url": "https://github.com/polytope-labs/hyperbridge" + } + ], + "database_specific": { + "cwe_ids": [ + "CWE-347" + ], + "severity": "CRITICAL", + "github_reviewed": true, + "github_reviewed_at": "2025-01-28T17:29:17Z", + "nvd_published_at": null + } +} \ No newline at end of file