Can you tell me whats wrong with this script?

This is a Gui that checks and gives the tool in a certain players name Do you know whats wrong with it?

local vip = {"Revenant101"}

function onButtonClicked()

local tools = { 

    game.Lighting.Crystal:clone(),

    game.Lighting.Memory:clone()
}
      for i, v in pairs(vip) do 

            if v:lower() == player.Name:lower() then 

                for t, o in pairs(tools) do 

                    o:clone().Parent = player.Backpack 

                end

            end

        end

end


script.Parent.MouseButton1Click:connect(onButtonClicked)