clearlinux/mixer-tools
2
0
Fork 0
mirror of https://github.com/clearlinux/mixer-tools.git synced 2026-09-06 21:51:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

swupd: accept Manifests with "contentsize: 0"

Browse Source 
Manifest.MoMs currently are produced with "contentsize: 0", e.g.
https://download.clearlinux.org/update/20010/Manifest.MoM.

Signed-off-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
This commit is contained in:
Caio Marcelo de Oliveira Filho
2018-01-05 16:16:56 -08:00
committed by tmarcu
parent ea2e2a1c05
commit b3cc6a108d
2 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
swupd/manifest.go
-4
View File
@@ -142,10 +142,6 @@ func (m *Manifest) CheckHeaderIsValid() error {
return errors.New("manifest has a zero file count")
}
if m.Header.ContentSize == 0 {
return errors.New("manifest has zero contentsize")
}
if m.Header.TimeStamp.IsZero() {
return errors.New("manifest timestamp not set")
}
swupd/manifest_test.go
-1
View File
@@ -216,7 +216,6 @@ func TestCheckInvalidManifestHeaders(t *testing.T) {
{"version zero", ManifestHeader{10, 0, 90, 553, time.Unix(1000, 0), 100000, nil}},
{"no files", ManifestHeader{10, 100, 90, 0, time.Unix(1000, 0), 100000, nil}},
{"no timestamp", ManifestHeader{10, 100, 90, 553, zeroTime, 100000, nil}},
{"zero contentsize", ManifestHeader{10, 100, 90, 553, time.Unix(1000, 0), 0, nil}},
{"version smaller than previous", ManifestHeader{10, 100, 110, 553, time.Unix(1000, 0), 100000, nil}},
}