TrueCloudLab/frostfs-node
18
1
Fork 27
Code Issues 91 Pull requests 8 Projects Releases 1 Activity Actions

[#951] adm: Check for error when reading contracts from archive
All checks were successful
Vulncheck / Vulncheck (pull_request) Successful in 3m17s
Details
DCO action / DCO (pull_request) Successful in 3m3s
Details
Build / Build Components (1.21) (pull_request) Successful in 4m28s
Details
Build / Build Components (1.22) (pull_request) Successful in 4m29s
Details
Pre-commit hooks / Pre-commit (pull_request) Successful in 4m46s
Details
Tests and linters / gopls check (pull_request) Successful in 6m41s
Details
Tests and linters / Lint (pull_request) Successful in 7m54s
Details
Tests and linters / Staticcheck (pull_request) Successful in 7m32s
Details
Tests and linters / Tests (1.21) (pull_request) Successful in 9m26s
Details
Tests and linters / Tests with -race (pull_request) Successful in 9m26s
Details
Tests and linters / Tests (1.22) (pull_request) Successful in 9m30s
Details

Browse source 
Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>
This commit is contained in:
Anton Nikiforov 2024-06-06 15:25:51 +03:00
parent 8fcd0f8f8d
commit efcebdf009
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
cmd/frostfs-adm/internal/modules/morph/helper/util.go
View file

@ -122,11 +122,11 @@ func readContractsFromArchive(file io.Reader, names []string) (map[string]*Contr
} }
r := tar.NewReader(gr) r := tar.NewReader(gr)
for h, err := r.Next(); ; h, err = r.Next() { var h *tar.Header
if err != nil { for h, err = r.Next(); err == nil && h != nil; h, err = r.Next() {
break if h.Typeflag != tar.TypeReg {
continue
} }
dir, _ := filepath.Split(h.Name) dir, _ := filepath.Split(h.Name)
ctrName := filepath.Base(dir) ctrName := filepath.Base(dir)
@ -149,6 +149,9 @@ func readContractsFromArchive(file io.Reader, names []string) (map[string]*Contr
} }
m[ctrName] = cs m[ctrName] = cs
} }
if err != nil && err != io.EOF {
return nil, fmt.Errorf("can't read contracts from archive: %w", err)
}
for ctrName, cs := range m { for ctrName, cs := range m {
if cs.RawNEF == nil { if cs.RawNEF == nil {