From 46199f2a16a0ebf2960488ee530b64ae9f71f5a6 Mon Sep 17 00:00:00 2001 From: rarified Date: Sat, 16 Oct 2021 10:27:34 -0600 Subject: [PATCH] #24 --- id/24/24/journal.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/id/24/24/journal.json b/id/24/24/journal.json index 88d4e44c..ce9c7604 100644 --- a/id/24/24/journal.json +++ b/id/24/24/journal.json @@ -1 +1 @@ -[{"date":"2021-10-12T19:20:07Z","author":"hoikas","fields":{"title":"Fix StrTo(Unicode|Ansi) Argument","body":"This fixes the incorrect size being passed to StrTo(Unicode|Ansi) in `VaultAgeFindOrCreateChildAgeLink`. `arrsize()` should only be used on arrays, not pointers - and those functions are expecting the size of the destination array, not the input string length.\n\nI noticed this when investigating a user report in Discord that their avatars are giving the \"Set Active Player Failed\" error. On examining the player vault, I found that the public city instance on MOULa has 133 child age instances of Descent. There are two problems with this. First, and most apparently, there should only be one child instance per Age. Second, the Descent child Age should be parented to Neighborhood, not city. What I suspect is happening is that the unsafe (and incorrect) usage of `arrsize` is evaluating to 1 and only the first character of the Age\u0027s name is being copied into the search, resulting in both the parent link search failing and the child link search failing.\n\nThis issue should not cause crashes on either the server or client side, but it may result in the proliferation of redundant child Ages in the vault and seemingly unexpected instance linking behavior.","type":"Bug","severity":"Unrated","priority":"Normal","mergeTo":"master","watchers":"+hoikas","status":"New"}},{"date":"2021-10-12T19:20:31Z","author":"hoikas","patchset":{"number":1,"rev":1,"tip":"d7c999c8f319a1b1a9d5de605a4fefb108821ba8","base":"3489b924cefee640d47591ef05b464c70142720e","insertions":2,"deletions":2,"commits":1,"added":0,"type":"Proposal"}}] \ No newline at end of file +[{"date":"2021-10-12T19:20:07Z","author":"hoikas","fields":{"title":"Fix StrTo(Unicode|Ansi) Argument","body":"This fixes the incorrect size being passed to StrTo(Unicode|Ansi) in `VaultAgeFindOrCreateChildAgeLink`. `arrsize()` should only be used on arrays, not pointers - and those functions are expecting the size of the destination array, not the input string length.\n\nI noticed this when investigating a user report in Discord that their avatars are giving the \"Set Active Player Failed\" error. On examining the player vault, I found that the public city instance on MOULa has 133 child age instances of Descent. There are two problems with this. First, and most apparently, there should only be one child instance per Age. Second, the Descent child Age should be parented to Neighborhood, not city. What I suspect is happening is that the unsafe (and incorrect) usage of `arrsize` is evaluating to 1 and only the first character of the Age\u0027s name is being copied into the search, resulting in both the parent link search failing and the child link search failing.\n\nThis issue should not cause crashes on either the server or client side, but it may result in the proliferation of redundant child Ages in the vault and seemingly unexpected instance linking behavior.","type":"Bug","severity":"Unrated","priority":"Normal","mergeTo":"master","watchers":"+hoikas","status":"New"}},{"date":"2021-10-12T19:20:31Z","author":"hoikas","patchset":{"number":1,"rev":1,"tip":"d7c999c8f319a1b1a9d5de605a4fefb108821ba8","base":"3489b924cefee640d47591ef05b464c70142720e","insertions":2,"deletions":2,"commits":1,"added":0,"type":"Proposal"}},{"date":"2021-10-16T16:27:34Z","author":"rarified","fields":{"status":"Open","watchers":"+rarified"}}] \ No newline at end of file