help defining player in module script?

local Roles = {}
local Players = game.Players:GetService("Players")
local PlayersService = game.Players

--Mafia Functions
Roles.Mafia = {}
function Roles.Mafia.Target(Player)
    local gui = Instance.new('ScreenGui')
    gui.Name = 'PlayerList'
    gui.Parent = Players.LocalPlayer:WaitForChild('PlayerGui')  

    local frm = Instance.new('ScrollingFrame')
    frm.Name = 'Content'
    frm.Size = UDim2.new(0, 200, 0, 400)
    frm.Position = UDim2.new(0.5, -180, 0.5, 170)
    frm.BackgroundColor3 = BrickColor.Black().Color
    frm.BackgroundTransparency = .7
    frm.Parent = gui

    local function mkLbl()
        local tmp = Instance.new('TextLabel')
        tmp.Size = UDim2.new(1,0, 0, 25)
        tmp.Font = Enum.Font.ArialBold
        tmp.FontSize = Enum.FontSize.Size18
        return tmp
    end

    local y = 0

    for i,v in pairs(game.Players:GetChildren()) do
        local tmp = mkLbl()
            tmp.Position = UDim2.new(0,0,0, y)
            y = y + 25 -- 25 size
            tmp.Text = Players.Name
            tmp.Parent = frm
            tmp.MouseButton1Click:connect(function()
                Player.Character:Destroy()
            end)
    end


end

See question.

Am I defining player correctly? Anything else wrong with function?