Blessed Sisturn II's buff doesn't account for players despawning

Got this in my server log:

Stale Component Reference: GUID 119393, scripts/prefabs/wendy.lua:141

So I checked, this bit here is missing an IsValid check:

	-- REMOVE PLAYERS NOW MISSING
	for _, player in ipairs(oldlist) do
		if player.components.sanity then
			local quit = true
			for _, newplayer in ipairs(newlist) do
				if player == newplayer then
					quit = false
					break
				end
			end
			if quit then
				local fx = SpawnPrefab("wendy_sanityaura_buff_off_fx")
				player.SoundEmitter:PlaySound("meta5/wendy/sisturn_sanity_buff_pst")
				player:AddChild(fx)
				player.components.sanity.neg_aura_modifiers:RemoveModifier(inst, "wendyskill"..inst.GUID)
			end
		end
	end

I'm surprised the SoundEmitter call doesn't cause a crash, but it does log that message which is good. Though what's worrying is the AddChild call, despite calling child.entity:SetParent(self.entity), it neither crashes nor logs anything..

Additionally, while the sanity modifier does get removed if the Wendy player despawns (due to the Wendy inst being used as the source), there's no accounting for the skill being deactivated (even if this really shouldn't happen normally), this means the buff could be stuck on players until Wendy despawns or the buffed players despawn. Also, players that lose the buff due to Wendy despawning will get no indication that they lost it.

Players in the list of sanityadjuster need to be processed to be unregistered when the skill is deactivated or if Wendy despawns.