feat: reject blank-name reward drops

Author: MikelCalvo

internal/minimal/shared_world.go

@@ -931,7 +931,7 @@ func (r *sharedWorldRegistry) RegisterGroundGold(ownerID uint64, ownerLogin stri
 }
 
 func (r *sharedWorldRegistry) registerGroundItem(ownerID uint64, ownerLogin string, character loginticket.Character, vid uint32, item inventory.ItemInstance, goldAmount uint32) bool {
-	if r == nil || ownerID == 0 || strings.TrimSpace(ownerLogin) == "" || vid == 0 || item.Vnum == 0 {
+	if r == nil || ownerID == 0 || strings.TrimSpace(ownerLogin) == "" || strings.TrimSpace(character.Name) == "" || vid == 0 || item.Vnum == 0 {
 		return false
 	}
 

internal/minimal/shared_world_test.go

@@ -491,6 +491,26 @@ func TestSharedWorldRegistryRegisterGroundRewardsRejectsBlankOwnerLogin(t *testi
 	}
 }
 
+func TestSharedWorldRegistryRegisterGroundRewardsRejectsBlankOwnerName(t *testing.T) {
+	registry := newSharedWorldRegistry()
+	owner := peerVisibilityCharacter("BlankNameRewardOwner", 0x010301a1, 0x020401a1, 1200, 2200, 0, 101, 201)
+	owner.Name = "\t"
+	ownerID, _ := registry.Join(owner, newPendingServerFrames(), nil)
+	if ownerID == 0 {
+		t.Fatal("expected blank-name reward owner join to allocate a shared-world entity id")
+	}
+
+	if registry.RegisterGroundItem(ownerID, "blank-name-reward-owner", owner, 0x07000021, inventory.ItemInstance{Vnum: 3001, Count: 1}) {
+		t.Fatal("expected blank-name ground-item reward registration to fail closed")
+	}
+	if registry.RegisterGroundGold(ownerID, "blank-name-reward-owner", owner, 0x07000022, 250) {
+		t.Fatal("expected blank-name ground-gold reward registration to fail closed")
+	}
+	if registry.GroundItemExists(0x07000021) || registry.GroundItemExists(0x07000022) {
+		t.Fatal("expected rejected blank-name reward ground entries to stay absent")
+	}
+}
+
 func TestSharedWorldRegistryRegisterGroundRewardsRejectsStaleLiveOwnerSnapshotAfterOwnerDeath(t *testing.T) {
 	registry := newSharedWorldRegistry()
 	owner := peerVisibilityCharacter("StaleLiveRewardOwner", 0x01030193, 0x02040193, 1200, 2200, 0, 101, 201)

spec/protocol/non-player-reward-bootstrap.md

@@ -57,6 +57,7 @@ Reward descriptors fail closed when:
 - any `reward_drop_vnums` entry is duplicated in the same descriptor
 - a runtime-generated ground-entry VID for a configured item-shaped or gold-shaped drop would be `0`
 - the reward owner login is empty or blank after trimming whitespace when registering a configured item-shaped or gold-shaped drop
+- the reward owner character name is empty or blank after trimming whitespace when registering a configured item-shaped or gold-shaped drop
 - a runtime-generated ground-item instance for a configured item-shaped drop would have `vnum = 0`
 - a runtime-generated ground-item instance for a configured item-shaped drop would have zero count
 - a runtime-generated ground-item instance for a configured item-shaped drop would exceed the current `GC ITEM_GET` count carrier (`255`)