Merge pull request #44 from andy89923/fix/term-panic

fix: check evtCh nil before ticker send to channel
free5gc · Apr 2, 2024 · 4a89dd2 · 4a89dd2
2 parents 705e50d + 6f29546
commit 4a89dd2
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/internal/forwarder/perio/server.go b/internal/forwarder/perio/server.go
@@ -64,9 +64,12 @@ func (pg *PERIOGroup) newTicker(wg *sync.WaitGroup, evtCh chan Event) error {
 			select {
 			case <-ticker.C:
 				logger.PerioLog.Debugf("ticker[%v] timeout", period)
-				evtCh <- Event{
-					eType:  TYPE_PERIO_TIMEOUT,
-					period: period,
+				// If the UPF had terminating, the evtCh would be nil
+				if evtCh != nil {
+					evtCh <- Event{
+						eType:  TYPE_PERIO_TIMEOUT,
+						period: period,
+					}
 				}
 			case <-pg.stopCh:
 				logger.PerioLog.Infof("ticker[%v] Stopped", period)