Fix RPM DB license handling (anchore#1184)

aiwantaozi · Oct 20, 2022 · 8e5e441 · 8e5e441
1 parent dbd4d42
commit 8e5e441
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/internal/formats/common/spdxhelpers/to_syft_model.go b/internal/formats/common/spdxhelpers/to_syft_model.go
@@ -322,13 +322,17 @@ func extractMetadata(p *spdx.Package2_2, info pkgInfo) (pkg.MetadataType, interf
 		} else {
 			epoch = &converted
 		}
+		license := p.PackageLicenseDeclared
+		if license == "" {
+			license = p.PackageLicenseConcluded
+		}
 		return pkg.RpmdbMetadataType, pkg.RpmdbMetadata{
 			Name:      p.PackageName,
 			Version:   p.PackageVersion,
 			Epoch:     epoch,
 			Arch:      arch,
 			SourceRpm: upstreamValue,
-			License:   p.PackageLicenseConcluded,
+			License:   license,
 			Vendor:    p.PackageOriginatorOrganization,
 		}
 	case pkg.DebPkg:

diff --git a/syft/pkg/cataloger/rpmdb/parse_rpmdb.go b/syft/pkg/cataloger/rpmdb/parse_rpmdb.go
@@ -92,6 +92,10 @@ func newPkg(resolver source.FilePathResolver, dbLocation source.Location, entry
 		Metadata:     metadata,
 	}
 
+	if entry.License != "" {
+		p.Licenses = append(p.Licenses, entry.License)
+	}
+
 	p.SetID()
 	return &p, nil
 }

diff --git a/syft/pkg/cataloger/rpmdb/parse_rpmdb_test.go b/syft/pkg/cataloger/rpmdb/parse_rpmdb_test.go
@@ -75,6 +75,7 @@ func TestParseRpmDB(t *testing.T) {
 					FoundBy:      catalogerName,
 					Type:         pkg.RpmPkg,
 					MetadataType: pkg.RpmdbMetadataType,
+					Licenses:     []string{"MIT"},
 					Metadata: pkg.RpmdbMetadata{
 						Name:      "dive",
 						Epoch:     nil,
@@ -102,6 +103,7 @@ func TestParseRpmDB(t *testing.T) {
 					FoundBy:      catalogerName,
 					Type:         pkg.RpmPkg,
 					MetadataType: pkg.RpmdbMetadataType,
+					Licenses:     []string{"MIT"},
 					Metadata: pkg.RpmdbMetadata{
 						Name:      "dive",
 						Epoch:     nil,