From fa17cbf985425505a3e79ae80553797556efbc60 Mon Sep 17 00:00:00 2001 From: "advisory-database[bot]" <45398580+advisory-database[bot]@users.noreply.github.com> Date: Mon, 28 Oct 2024 18:29:15 +0000 Subject: [PATCH] Publish GHSA-ghjw-32xw-ffwr --- .../GHSA-ghjw-32xw-ffwr.json | 77 +++++++++++++++++++ 1 file changed, 77 insertions(+) create mode 100644 advisories/github-reviewed/2024/10/GHSA-ghjw-32xw-ffwr/GHSA-ghjw-32xw-ffwr.json diff --git a/advisories/github-reviewed/2024/10/GHSA-ghjw-32xw-ffwr/GHSA-ghjw-32xw-ffwr.json b/advisories/github-reviewed/2024/10/GHSA-ghjw-32xw-ffwr/GHSA-ghjw-32xw-ffwr.json new file mode 100644 index 00000000000..db6e944c598 --- /dev/null +++ b/advisories/github-reviewed/2024/10/GHSA-ghjw-32xw-ffwr/GHSA-ghjw-32xw-ffwr.json @@ -0,0 +1,77 @@ +{ + "schema_version": "1.4.0", + "id": "GHSA-ghjw-32xw-ffwr", + "modified": "2024-10-28T18:28:03Z", + "published": "2024-10-28T18:28:03Z", + "aliases": [ + "CVE-2024-47827" + ], + "summary": "Argo Workflows Controller: Denial of Service via malicious daemon Workflows", + "details": "### Summary\n\nDue to a race condition in a global variable, the argo workflows controller can be made to crash on-command by any user with access to execute a workflow.\n\nThis was resolved by https://github.com/argoproj/argo-workflows/pull/13641\n\n### Details\n\nThese two lines introduce a data race in the underlying SPDY implementation of the Kubernetes API client. If a second request is made before the first completes, it results in a panic due to a null pointer.\n* https://github.com/argoproj/argo-workflows/blob/ce7f9bfb9b45f009b3e85fabe5e6410de23c7c5f/workflow/metrics/metrics_k8s_request.go#L49\n* https://github.com/argoproj/argo-workflows/blob/ce7f9bfb9b45f009b3e85fabe5e6410de23c7c5f/workflow/metrics/metrics_k8s_request.go#L75\n\nThis appears to have been added in this commit https://github.com/argoproj/argo-workflows/commit/9756babd0ed589d1cd24592f05725f748f74130b / #13265 / v3.6.0-rc1\n\n### PoC\n\nWith the `KUBECONFIG` variable set to an appropriate file with `create` permissions for the `Workflow` kind, execute the following bash script:\n\n```bash\n#!/bin/bash -xeu\n\nwhile true ; do\n name=$(\n { argo submit /dev/stdin <<'EOF'\napiVersion: argoproj.io/v1alpha1\nkind: Workflow\nmetadata:\n generateName: curl-\nspec:\n entrypoint: main\n templates:\n - name: main\n dag:\n tasks:\n - name: no-op\n template: no-op\n withSequence:\n count: 3\n - name: no-op\n daemon: true\n container:\n image: alpine:3.13\n command: [sleep, infinity]\nEOF\n } | head -n1 | awk '{ print $2 }'\n )\n ( sleep 30; argo terminate $name ) &\n sleep 15\ndone\n```\n\nThis script creates, and subsequently cleans up, multiple `daemon` pods in rapid succession. Each pod cleanup involves executing a `kill` instruction using the Kubernetes `exec` API, triggering the conditions for the panic. This can be seen when the tests mark the pods as complete, but the workflow itself never completes. Observing the controller logs when this happens shows the panic and restart of the controller every few seconds. In a setup with exponential backoff (e.g. a Kubernetes Pod) this is enough to reliably cause crashes enough to extend this backoff significantly and leave other workflows stalled.\n\nBecause the restarted controller believes it has sent the `kill` signal, it will wait indefinitely for the pod to terminate, which it never will, so the attack must constantly garbage-collect its own workflows with the `argo terminate` command, otherwise the maximum concurrently running workflows will be reached. A more sophisticated attack could detect when the workflow has been signaled to clean up and terminate it then instead of relying on a simple timer.\n\n### Impact\n\nA malicious user with access to create workflows can continually submit workflows that do nothing except create and then clean up multiple daemon pods, resulting in a crash-loop that prevents other users' workflows from running. This can be done with only a handful of pods and very little cpu and memory, meaning typical multi-tenant Kubernetes controls such as Pod count and resource quotas are not effective at preventing it.\n\nBecause the panic log does not in any way suggest that the issue has anything to do with the daemon pods, and an attacker could easily disguise these daemon pods as part of a genuine workflow, it would be difficult for administrators to discover the root cause of the DoS and the individuals responsible to remove their access.\n", + "severity": [ + { + "type": "CVSS_V3", + "score": "CVSS:3.1/AV:A/AC:L/PR:L/UI:N/S:U/C:N/I:N/A:H" + }, + { + "type": "CVSS_V4", + "score": "CVSS:4.0/AV:A/AC:L/AT:N/PR:L/UI:N/VC:N/VI:N/VA:H/SC:N/SI:N/SA:N" + } + ], + "affected": [ + { + "package": { + "ecosystem": "Go", + "name": "github.com/argoproj/argo-workflows/v3" + }, + "ranges": [ + { + "type": "ECOSYSTEM", + "events": [ + { + "introduced": "3.6.0-rc1" + }, + { + "fixed": "3.6.0-rc2" + } + ] + } + ], + "versions": [ + "3.6.0-rc1" + ] + } + ], + "references": [ + { + "type": "WEB", + "url": "https://github.com/argoproj/argo-workflows/security/advisories/GHSA-ghjw-32xw-ffwr" + }, + { + "type": "WEB", + "url": "https://github.com/argoproj/argo-workflows/pull/13641" + }, + { + "type": "WEB", + "url": "https://github.com/argoproj/argo-workflows/commit/524406451f4dfa57bf3371fb85becdb56a2b309a" + }, + { + "type": "PACKAGE", + "url": "https://github.com/argoproj/argo-workflows" + }, + { + "type": "WEB", + "url": "https://github.com/argoproj/argo-workflows/blob/ce7f9bfb9b45f009b3e85fabe5e6410de23c7c5f/workflow/metrics/metrics_k8s_request.go#L75" + } + ], + "database_specific": { + "cwe_ids": [ + "CWE-1108", + "CWE-362" + ], + "severity": "MODERATE", + "github_reviewed": true, + "github_reviewed_at": "2024-10-28T18:28:03Z", + "nvd_published_at": null + } +} \ No newline at end of file