Anti-Speed Hacker Script Not Working?

local MaxSpeed = 16 -- Max speed possible in game

-- Note: I've added a rule-counter, which means you have to go too fast for 2 ticks to get punished.
-- That's necessary for teleporting by spawning, maybe your admins tp'ing, entering a vehicle, ...

local caught = {}
local function punish(plr)
    if caught[plr] then
        -- More than 2x too fast in a second? Seems like he needs to get punished
        -- (As far as I know, it's impossible to teleport 2x in a second without exploiting)
        -- (Assuming that people don't use admincommands teleport/ very fast)
        -- (Also assuming that the game doesn't teleport someone 2x in a second)
        if tick() - caught[plr] < 1 then
            -- Should never fail, as we don't check RobloxLocked players
            -- Even if it fails, punish() is called in a pcall() so it's safe anyway
            plr:Kick()
            print(plr,"kicked for speed hacking") -- Print in Server Console
        end
    end caught[plr] = tick()
end

-- We don't want to flood the caught table with offline exploiters who were here, don't we?
game:GetService("Players").PlayerRemoving:connect(function(p)
    caught[p] = nil -- remove old unneeded data
end)

local function ignoreState(state)
    if state == Enum.HumanoidStateType.Running then
        return false
    elseif state == Enum.HumanoidStateType.RunningNoPhysics then
        return false
    elseif state == Enum.HumanoidStateType.StrafingNoPhysics then
        return false
    elseif state == Enum.HumanoidStateType.Climbing then
        return false
    elseif state == Enum.HumanoidStateType.RunningNoPhysics then
        return false
    end return true
end

while true do
    --[[ Saving the current position (position just before the tick change) ]]--
    local pos = {}
    for k,v in pairs(game:GetService("Players"):GetPlayers()) do
        if v.Character and v.Character:findFirstChild("Torso") then
            -- Renamed/Absent Humanoid? Oh well, can't predict that
            if not v.Character:findFirstChild("Humanoid") then return end
            local human = v.Character.Humanoid -- need to use it a lot
            -- Torso can be flinged away on dead, making false results
            if human.Health == 0 then return end
            -- If he sits, we assume he's in a vehicle, thuss may go faster
            if human.Sit then return end
            -- If he's platformstanding, we assume he's skating or so
            if human.PlatformStand then return end
            -- only check if the player is actually running
            if ignoreState(human:GetState()) then return end
            -- Save current position, for comparing during the next loop
            pos[v] = v.Character.Torso.Position
        end
    end

    --[[ Max Distance Calculation ]]--
    local tim = wait() -- Get how long the last tick happened
    local dis = (MaxSpeed + 2) * tim -- Get the max distance someone can move in that time interval
    -- (MaxSpeed + 2): 2 being a safe margin. (Weird calculation? Jumping while running?)

    for k,v in pairs(pos) do -- Time to see if he went too fast
        pcall(function()
            if v.Character and v.Character:findFirstChild("Torso") then
                if not v.Character:findFirstChild("Humanoid") then return end
                local human = v.Character.Humanoid
                if human.Health == 0 then return end
                if human.PlatformStand then return end
                if human.Sit then return end
                if ignoreState(human:GetState()) then return end
                -- Yeuy! Get the distance calculated!
                local d = (pos[v] - v.Character.Torso.Position).magnitude
                if d > dis then pcall(punish,v) end
            end
        end)
    end

    wait(0.1) -- Just to make this a bit more performance-friendly
    -- (Not that it's very unfriendly, though, unless there are 100 players)
end