commit c4b19116940e73edfe9d49c05a9551d361d6a844
author Jonathan Hart <jono@opennetworking.org> Thu Apr 02 11:21:45 2020 -0700
committer Jonathan Hart <jono@opennetworking.org> Thu Apr 02 11:35:14 2020 -0700
tree 4e82591a3a820b9784a8bf96b22483c8950739f3
parent 38d533d9e90cd72e47ea4460f021694c7937281f

Avoid crash when trying to log nil group.Desc

Change-Id: I0c7bafa98e406ba4a0f3b61c6554bd167151835a

internal/pkg/core/openolt_flowmgr.go

diff --git a/internal/pkg/core/openolt_flowmgr.go b/internal/pkg/core/openolt_flowmgr.go
index 3522ebd..b8875e7 100644
--- a/internal/pkg/core/openolt_flowmgr.go
+++ b/internal/pkg/core/openolt_flowmgr.go
@@ -2033,7 +2033,7 @@
 func (f *OpenOltFlowMgr) ModifyGroup(ctx context.Context, group *ofp.OfpGroupEntry) error {
 	logger.Infow("modify-group", log.Fields{"group": group})
 	if group == nil || group.Desc == nil {
-		return olterrors.NewErrInvalidValue(log.Fields{"group": group, "groupDesc": group.Desc}, nil)
+		return olterrors.NewErrInvalidValue(log.Fields{"group": group}, nil)
 	}
 
 	newGroup := f.buildGroup(group.Desc.GroupId, group.Desc.Buckets)