aboutsummaryrefslogtreecommitdiffstats
path: root/pkg/covermerger
diff options
context:
space:
mode:
authorTaras Madan <tarasmadan@google.com>2024-08-21 12:42:58 +0200
committerTaras Madan <tarasmadan@google.com>2024-08-22 09:08:20 +0000
commitaa99fc3349e97ea596c31624efde306de4136241 (patch)
tree8ec3d255838cd3990170bd33bfc0d641c0aac418 /pkg/covermerger
parentca02180f7c9d6b3a7de8a887f3998725ae2f0c51 (diff)
pkg/validator: initial code
Diffstat (limited to 'pkg/covermerger')
-rw-r--r--pkg/covermerger/bq_csv_reader.go16
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"