VYPR
High severityNVD Advisory· Published Feb 21, 2024· Updated Aug 14, 2024

Helm's Missing YAML Content Leads To Panic

CVE-2024-26147

Description

Helm is a package manager for Charts for Kubernetes. Versions prior to 3.14.2 contain an uninitialized variable vulnerability when Helm parses index and plugin yaml files missing expected content. When either an index.yaml file or a plugins plugin.yaml file were missing all metadata a panic would occur in Helm. In the Helm SDK, this is found when using the LoadIndexFile or DownloadIndexFile functions in the repo package or the LoadDir function in the plugin package. For the Helm client this impacts functions around adding a repository and all Helm functions if a malicious plugin is added as Helm inspects all known plugins on each invocation. This issue has been resolved in Helm v3.14.2. If a malicious plugin has been added which is causing all Helm client commands to panic, the malicious plugin can be manually removed from the filesystem. If using Helm SDK versions prior to 3.14.2, calls to affected functions can use recover to catch the panic.

Affected packages

Versions sourced from the GitHub Security Advisory.

PackageAffected versionsPatched versions
helm.sh/helm/v3Go
< 3.14.23.14.2

Affected products

1

Patches

1
bb4cc9125503

Merge pull request from GHSA-r53h-jv2g-vpx6

https://github.com/helm/helmMatt FarinaFeb 21, 2024via ghsa
4 files changed · +18 0
  • pkg/plugin/plugin.go+4 0 modified
    @@ -175,6 +175,10 @@ var validPluginName = regexp.MustCompile("^[A-Za-z0-9_-]+$")
     
     // validatePluginData validates a plugin's YAML data.
     func validatePluginData(plug *Plugin, filepath string) error {
    +	// When metadata section missing, initialize with no data
    +	if plug.Metadata == nil {
    +		plug.Metadata = &Metadata{}
    +	}
     	if !validPluginName.MatchString(plug.Metadata.Name) {
     		return fmt.Errorf("invalid plugin name at %q", filepath)
     	}
    
  • pkg/plugin/plugin_test.go+6 0 modified
    @@ -353,6 +353,11 @@ func TestSetupEnvWithSpace(t *testing.T) {
     }
     
     func TestValidatePluginData(t *testing.T) {
    +	// A mock plugin missing any metadata.
    +	mockMissingMeta := &Plugin{
    +		Dir: "no-such-dir",
    +	}
    +
     	for i, item := range []struct {
     		pass bool
     		plug *Plugin
    @@ -363,6 +368,7 @@ func TestValidatePluginData(t *testing.T) {
     		{false, mockPlugin("$foo -bar")}, // Test leading chars
     		{false, mockPlugin("foo -bar ")}, // Test trailing chars
     		{false, mockPlugin("foo\nbar")},  // Test newline
    +		{false, mockMissingMeta},         // Test if the metadata section missing
     	} {
     		err := validatePluginData(item.plug, fmt.Sprintf("test-%d", i))
     		if item.pass && err != nil {
    
  • pkg/repo/index.go+4 0 modified
    @@ -359,6 +359,10 @@ func loadIndex(data []byte, source string) (*IndexFile, error) {
     				log.Printf("skipping loading invalid entry for chart %q from %s: empty entry", name, source)
     				continue
     			}
    +			// When metadata section missing, initialize with no data
    +			if cvs[idx].Metadata == nil {
    +				cvs[idx].Metadata = &chart.Metadata{}
    +			}
     			if cvs[idx].APIVersion == "" {
     				cvs[idx].APIVersion = chart.APIVersionV1
     			}
    
  • pkg/repo/index_test.go+4 0 modified
    @@ -70,6 +70,10 @@ entries:
         name: grafana
       foo:
       -
    +  bar:
    +  - digest: "sha256:1234567890abcdef"
    +    urls:
    +    - https://charts.helm.sh/stable/alpine-1.0.0.tgz
     `
     )
     
    

Vulnerability mechanics

Generated by null/stub on May 9, 2026. Inputs: CWE entries + fix-commit diffs from this CVE's patches. Citations validated against bundle.

References

4

News mentions

0

No linked articles in our index yet.