diff options
| author | Taras Madan <tarasmadan@google.com> | 2024-08-21 12:42:58 +0200 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2024-08-22 09:08:20 +0000 |
| commit | aa99fc3349e97ea596c31624efde306de4136241 (patch) | |
| tree | 8ec3d255838cd3990170bd33bfc0d641c0aac418 /pkg/covermerger | |
| parent | ca02180f7c9d6b3a7de8a887f3998725ae2f0c51 (diff) | |
pkg/validator: initial code
Diffstat (limited to 'pkg/covermerger')
| -rw-r--r-- | pkg/covermerger/bq_csv_reader.go | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/pkg/covermerger/bq_csv_reader.go b/pkg/covermerger/bq_csv_reader.go index 68711e318..60f6e5829 100644 --- a/pkg/covermerger/bq_csv_reader.go +++ b/pkg/covermerger/bq_csv_reader.go @@ -8,20 +8,14 @@ import ( "context" "fmt" "io" - "regexp" "cloud.google.com/go/bigquery" "cloud.google.com/go/civil" "github.com/google/syzkaller/pkg/gcs" + "github.com/google/syzkaller/pkg/validator" "github.com/google/uuid" ) -var allowedFilePath = regexp.MustCompile(`^[./_a-zA-Z0-9]*$`) - -func isAllowedFilePath(s string) bool { - return allowedFilePath.MatchString(s) -} - type bqCSVReader struct { closers []io.Closer gcsFiles []io.Reader @@ -38,8 +32,12 @@ func MakeBQCSVReader() *bqCSVReader { } func (r *bqCSVReader) InitNsRecords(ctx context.Context, ns, filePath, commit string, from, to civil.Date) error { - if !isAllowedFilePath(filePath) { - return fmt.Errorf("wrong file path '%s'", filePath) + if err := validator.AnyError("input validation failed", + validator.NamespaceName(ns), + validator.KernelFilePath(filePath), + validator.AnyOk(validator.EmptyStr(commit), validator.CommitHash(commit)), + ); err != nil { + return err } sessionUUID := uuid.New().String() gsBucket := "syzbot-temp" |
