lightningd: fix bogus channel iteration on setchannelfee

Setting channel to NULL then iterating doesn't work!

Fixes: #2733
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

lightningd/peer_control.c

@@ -1537,15 +1537,13 @@ static struct command_result *json_setchannelfee(struct command *cmd,
 	/* If the users requested 'all' channels we need to iterate */
 	if (channel == NULL) {
 		list_for_each(&cmd->ld->peers, peer, list) {
-			list_for_each(&peer->channels, channel, list) {
+			channel = peer_active_channel(peer);
-				channel = peer_active_channel(peer);
+			if (!channel)
-				if (!channel)
+				continue;
-					continue;
+			if (channel->state != CHANNELD_NORMAL &&
-				if (channel->state != CHANNELD_NORMAL &&
+			    channel->state != CHANNELD_AWAITING_LOCKIN)
-					channel->state != CHANNELD_AWAITING_LOCKIN)
+				continue;
-					continue;
+			set_channel_fees(cmd, channel, *base, *ppm, response);
-				set_channel_fees(cmd, channel, *base, *ppm, response);
-			}
 		}
 
 	/* single channel should be updated */