chore: minor audit fixes (#227)

* add len

* format

* map cap

* typo

* typo

* typo

pkg/math/median/median.go

@@ -36,8 +36,7 @@ func ComputeMedian() types.PriceAggregationFn {
 			}
 		}
 
-		medianPrices := make(types.TickerPrices)
-
+		medianPrices := make(types.TickerPrices, len(pricesByAsset))
 		// Iterate through all assets and compute the median price
 		for cp, prices := range pricesByAsset {
 			if len(prices) == 0 {

pkg/math/oracle/median.go

@@ -44,7 +44,7 @@ func NewMedianAggregator(logger *zap.Logger, cfg mmtypes.MarketMap) (*MedianAggr
 	}, nil
 }
 
-// AggregatedData implements the aggregate function for the median price calculation. Specifically, this
+// AggregateData implements the aggregate function for the median price calculation. Specifically, this
 // aggregation function aggregates the prices seen by each provider by first converting each price to a
 // common ticker and then calculating the median of the converted prices. Prices are converted either
 //

x/oracle/types/genesis.go

@@ -38,7 +38,7 @@ func DefaultGenesisState() *GenesisState {
 }
 
 // Validate validates the currency-pair geneses that the Genesis-State is composed of
-// valid CurrencyPairGeneses, and that no ID for a currency-pair is repeated.
+// valid CurrencyPairGenesis, and that no ID for a currency-pair is repeated.
 func (gs *GenesisState) Validate() error {
 	ids := make(map[uint64]struct{})
 	cps := make(map[string]struct{})
@@ -48,7 +48,7 @@ func (gs *GenesisState) Validate() error {
 			return err
 		}
 
-		// check if the ID > that gs.NextID
+		// check if the ID > gs.NextID
 		if cpg.Id >= gs.NextId {
 			return fmt.Errorf("invalid id: %v, must be less than next id: %v", cpg.Id, gs.NextId)
 		}

x/sla/keeper/msg_server.go

@@ -39,7 +39,7 @@ func (m *MsgServer) AddSLAs(goCtx context.Context, req *slatypes.MsgAddSLAs) (*s
 }
 
 // RemoveSLAs defines a method that removes a set of SLAs from the store. The SLAs provided must
-// do not need to exist in the store, and the signer of the message must be the module authority.
+// exist in the store, and the signer of the message must be the module authority.
 func (m *MsgServer) RemoveSLAs(goCtx context.Context, req *slatypes.MsgRemoveSLAs) (*slatypes.MsgRemoveSLAsResponse, error) {
 	// Remove the SLAs in message from state.
 	ctx := sdk.UnwrapSDKContext(goCtx)