add skins limit value to settings, sanitize skin description

init.lua

@@ -19,7 +19,8 @@ skins = {
 	id = 1,
 	file = minetest.get_worldpath() .. "/simple_skins.mt",
 	preview = minetest.settings:get_bool("simple_skins_preview"),
-	translate = S
+	translate = S,
+	skin_limit = tonumber(minetest.settings:get("simple_skins_limit")) or 300
 }
 
 
@@ -41,7 +42,7 @@ end
 -- load skin list and metadata
 local f, data, skin = 1
 
-while true do
+while skins.id <= skins.skin_limit do
 
 	skin = "character_" .. skins.id
 
@@ -69,8 +70,8 @@ while true do
 
 	-- add metadata to list
 	skins.meta[skin] = {
-		name = data and data.name or "",
-		author = data and data.author or ""
+		name = data and data.name and data.name:gsub("[%p%c]", "") or "",
+		author = data and data.author and data.author:gsub("[%p%c]", "") or ""
 	}
 
 	skins.id = skins.id + 1

readme.md

@@ -7,6 +7,7 @@ https://forum.minetest.net/viewtopic.php?id=9100
 
 Change log:
 
+- 1.0 - Added skin_limit setting to limit skins loaded, sanitized skin description
 - 0.9 - Added Unified Inventory support (thanks Opvolger)
 - 0.8 - Added player model preview when viewing formspec (Minetest 5.4 dev only)
 - 0.7 - Add some error checks, improve /setskin and tweak & tidy code

settingtypes.txt

@@ -1,2 +1,5 @@
 #    When true character model will appear in formspec with current skin as texture
 simple_skins_preview (Simple Skins Preview) bool false
+
+#    Skin limit number to limit the number of skins read onto list
+simple_skins_limit (Skin Limit) int 300