validate quorum ID

Layr-Labs · Jun 20, 2024 · ab28b89 · ab28b89
1 parent c5de108
commit ab28b89
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/disperser/dataapi/subgraph_client.go b/disperser/dataapi/subgraph_client.go
@@ -458,6 +458,9 @@ func parseOperatorQuorum(operatorQuorum []*subgraph.OperatorQuorum) ([]*Operator
 			if err != nil {
 				return nil, err
 			}
+			if quorum < 0 || 255 < quorum {
+				return nil, fmt.Errorf("Given quorum ID %d is not between 0 and 255", quorum)
+			}
 			quorumNumbers = append(quorumNumbers, uint8(quorum))
 		}
 		parsed[i] = &OperatorQuorum{

diff --git a/node/config.go b/node/config.go
@@ -93,6 +93,9 @@ func NewConfig(ctx *cli.Context) (*Config, error) {
 		if err != nil {
 			return nil, err
 		}
+		if val < 0 || 255 < val {
+			return nil, fmt.Errorf("Given quorum ID %d is not between 0 and 255", val)
+		}
 		ids = append(ids, core.QuorumID(val))
 	}
 	if len(ids) == 0 {

diff --git a/node/plugin/config.go b/node/plugin/config.go
@@ -2,6 +2,7 @@ package plugin
 
 import (
 	"errors"
+	"fmt"
 	"strconv"
 	"strings"
 
@@ -136,6 +137,9 @@ func NewConfig(ctx *cli.Context) (*Config, error) {
 		if err != nil {
 			return nil, err
 		}
+		if val < 0 || 255 < val {
+			return nil, fmt.Errorf("Given quorum ID %d is not between 0 and 255", val)
+		}
 		ids = append(ids, core.QuorumID(val))
 	}
 	if len(ids) == 0 {