diff options
author | specter25 <ujjwalcoding012@gmail.com> | 2021-03-28 01:04:16 +0530 |
---|---|---|
committer | specter25 <ujjwalcoding012@gmail.com> | 2021-03-28 01:04:16 +0530 |
commit | 07cb513803ea144c9ca427b413c9ac89f6b8006d (patch) | |
tree | 0e7849383ba8cc5674c93aad150ddddc0b6dcbd9 /tvloader/parser2v1/parse_package.go | |
parent | 240e16abc6fa09ba7a2a138f5b8e258052bef856 (diff) | |
download | spdx-tools-07cb513803ea144c9ca427b413c9ac89f6b8006d.tar.gz |
Error on missing package and file ids in v2.1
- Shows error if a Package has a missing SPDXID tag
- Show error if a File has a missing SPDXID tag
Signed-off-by: specter25 <ujjwalcoding012@gmail.com>
Diffstat (limited to 'tvloader/parser2v1/parse_package.go')
-rw-r--r-- | tvloader/parser2v1/parse_package.go | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tvloader/parser2v1/parse_package.go b/tvloader/parser2v1/parse_package.go index b653d9e..9a03e9a 100644 --- a/tvloader/parser2v1/parse_package.go +++ b/tvloader/parser2v1/parse_package.go @@ -20,6 +20,10 @@ func (parser *tvParser2_1) parsePairFromPackage2_1(tag string, value string) err case "PackageName": // if package already has a name, create and go on to a new package if parser.pkg == nil || parser.pkg.PackageName != "" { + //check if the previous package contained a spdxId or not + if parser.pkg != nil && parser.pkg.PackageSPDXIdentifier == spdx.ElementID("") { + return fmt.Errorf("Invalid package without a package SPDX identifier") + } parser.pkg = &spdx.Package2_1{ FilesAnalyzed: true, IsFilesAnalyzedTagPresent: false, |