Fix controller nil pointer deref

Solar-Punk-Ltd · May 13, 2024 · 81e3b04 · 81e3b04
1 parent 7960a49
commit 81e3b04
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/pkg/dynamicaccess/controller.go b/pkg/dynamicaccess/controller.go
@@ -162,6 +162,15 @@ func (c *controller) HandleGrantees(
 		if err != nil {
 			return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
 		}
+		// generate new access key and new act
+		act, err = kvs.New(ls)
+		if err != nil {
+			return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
+		}
+		err = c.accessLogic.AddPublisher(ctx, act, publisher)
+		if err != nil {
+			return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
+		}
 	}
 
 	var gl GranteeList
@@ -195,17 +204,15 @@ func (c *controller) HandleGrantees(
 	}
 
 	var granteesToAdd []*ecdsa.PublicKey
-	// generate new access key and new act
 	if len(removeList) != 0 || encryptedglref.IsZero() {
+		// generate new access key and new act
 		act, err = kvs.New(ls)
 		if err != nil {
 			return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
 		}
-		if historyref.IsZero() {
-			err = c.accessLogic.AddPublisher(ctx, act, publisher)
-			if err != nil {
-				return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
-			}
+		err = c.accessLogic.AddPublisher(ctx, act, publisher)
+		if err != nil {
+			return swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, swarm.ZeroAddress, err
 		}
 		granteesToAdd = gl.Get()
 	} else {