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:)