Originally Posted by thomasjohnshannon
Has anyone else noticed an error when you /camp if you have an addon with a SlashCmdList loaded?
|
Yes. They forgot to add the slash handlers for logout and quit to the secure command hash table. The table is populated with all the existing SecureCmdList functions, but logout and quit are declared afterwards with the rest of the unprotected functions.
Lua Code:
-- starting at line 1584
-- Pre-populate the secure command hash table
for index, value in pairs(SecureCmdList) do
local i = 1;
local cmdString = _G["SLASH_"..index..i];
while ( cmdString ) do
cmdString = strupper(cmdString);
hash_SecureCmdList[cmdString] = value; -- add to hash
i = i + 1;
cmdString = _G["SLASH_"..index..i];
end
end
Lua Code:
-- starting at line 1700
SlashCmdList["LOGOUT"] = function(msg)
Logout();
end
SlashCmdList["QUIT"] = function(msg)
if (IsKioskModeEnabled()) then
return;
end
Quit();
end
This should be reported as a bug.