Confuse of ClassB Parameters set

[redwolf2019]

File location:chirpstack-network-server/internal/data/data.go

Source code:

func setPingSlotParameters(ctx *dataContext) error {
	if !ctx.DeviceProfile.SupportsClassB {
		return nil
	}

	if classBPingSlotDR != ctx.DeviceSession.PingSlotDR || classBPingSlotFrequency != ctx.DeviceSession.PingSlotFrequency {
		block := maccommand.RequestPingSlotChannel(ctx.DeviceSession.DevEUI, classBPingSlotDR, classBPingSlotFrequency)
		ctx.MACCommands = append(ctx.MACCommands, block)
	}

	return nil
}

Bug:

if classBPingSlotDR != ctx.DeviceSession.PingSlotDR || classBPingSlotFrequency != ctx.DeviceSession.PingSlotFrequency {
		block := maccommand.RequestPingSlotChannel(ctx.DeviceSession.DevEUI, classBPingSlotDR, classBPingSlotFrequency)
		ctx.MACCommands = append(ctx.MACCommands, block)
}

According to my opinion,PingSlotDR/classBPingSlotFrequency always loading from chirpstack-network-server.toml, not loading from ctx.DeviceSession.

My Patch is:

        var classBPingSlotDRReal int
	if classBPingSlotDR != ctx.DeviceSession.PingSlotDR {
		classBPingSlotDRReal = ctx.DeviceSession.PingSlotDR
	}else {
		classBPingSlotDRReal = classBPingSlotDR
	}

	var classBPingSlotFrequencyReal uint32
	if classBPingSlotFrequency != ctx.DeviceSession.PingSlotFrequency {
		classBPingSlotFrequencyReal = ctx.DeviceSession.PingSlotFrequency
	}else {
		classBPingSlotFrequencyReal = classBPingSlotFrequency
	}

	block := maccommand.RequestPingSlotChannel(ctx.DeviceSession.DevEUI, classBPingSlotDRReal, classBPingSlotFrequencyReal)
	ctx.MACCommands = append(ctx.MACCommands, block)

Is my idea correct? Looking forward to your reply:)