check measurePeriod before apply URR (#33)

free5gc · Jun 8, 2023 · 4474dc8 · 4474dc8
1 parent e623954
commit 4474dc8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/internal/forwarder/gtp5g.go b/internal/forwarder/gtp5g.go
@@ -1132,6 +1132,9 @@ func (g *Gtp5g) CreateURR(lSeid uint64, req *ie.IE) error {
 			if err != nil {
 				return err
 			}
+			if measurePeriod <= 0 {
+				return errors.New("invalid measurement period")
+			}
 			// TODO: convert time.Duration -> ?
 			attrs = append(attrs, nl.Attr{
 				Type:  gtp5gnl.URR_MEASUREMENT_PERIOD,
@@ -1168,6 +1171,9 @@ func (g *Gtp5g) CreateURR(lSeid uint64, req *ie.IE) error {
 	}
 
 	if rptTrig.PERIO() {
+		if measurePeriod <= 0 {
+			return errors.New("invalid measurement period")
+		}
 		g.ps.AddPeriodReportTimer(lSeid, urrid, measurePeriod)
 	}