OldUnreal (Engine)

That memory error is simply caused by this:

Code: Select all

var PlayerReplicationInfo MsgPlayer[64];

Pointer array to PlayerReplicationInfo actors. When some player have said a chat message it stores it there in the list, and when player disconnects from server the actor cleanup code dosen't set it's reference to NULL (it only NULL's out references from current in level Actors, not ALL objects) leaving an invalid pointer to that PRI. So next time you bring down the console and it tries to render chatting player name it crashes from that invalid pointer in this line:

Code: Select all

simulated function DrawConsoleView( Canvas C )
{
...
            if (( MsgType[Line] == 'Say' ) || ( MsgType[Line] == 'TeamSay' ))
                  C.StrLen( MsgPlayer[Line].PlayerName$":"@MsgText[Line], XL, YL );

But indeed its fixable by setting it work alike UWindow console (store playername instead of PRI).

That memory error is simply caused by this:

Code: Select all

var PlayerReplicationInfo MsgPlayer[64];

Pointer array to PlayerReplicationInfo actors. When some player have said a chat message it stores it there in the list, and when player disconnects from server the actor cleanup code dosen't set it's reference to NULL (it only NULL's out references from current in level Actors, not ALL objects) leaving an invalid pointer to that PRI. So next time you bring down the console and it tries to render chatting player name it crashes from that invalid pointer in this line:

Code: Select all

simulated function DrawConsoleView( Canvas C )
{
...
            if (( MsgType[Line] == 'Say' ) || ( MsgType[Line] == 'TeamSay' ))
                  C.StrLen( MsgPlayer[Line].PlayerName$":"@MsgText[Line], XL, YL );

But indeed its fixable by setting it work alike UWindow console (store playername instead of PRI).

Shouldn't that give some form of NULL pointer error? The crash I've always gotten was "Ran out of virtual memory. To fix this issue...". Are you talking about some other crash that I've just never seen?

Actually it is possible it tries to allocates too long string when it duplicates the string for the function inner use.
As for the mapchange, the console script clears the message list so it wouldnt crash from that (as this would obviously leave old maps referenced in memory):

Code: Select all

event NotifyLevelChange()
{
...
      ClearMessages();
}

function ClearMessages()
{
      for (i=0; i

And the only way to change the type to Say or TeamSay would overwrite any invalid PRI reference. I really don't think that's the place where the error occurs.

And the only way to change the type to Say or TeamSay would overwrite any invalid PRI reference. I really don't think that's the place where the error occurs.

No, you dont see the error. You see in online games while ur playing in some server you receive a lot of chat messages, then the chatting player disconnects away from server (deleting PRI Actor after a while but still leaving in Console reference to it). So next time you bring down console (while being in that server), your game may crash (depending on whatever actor has been cleaned up from memory yet or only have bDeleteMe on True).

Thats because destroyed actors will stay in memory until deleted actors chain is big enough and it will cleanup them from memory (it has nothing to do with garbage collecting), however while cleaning up from memory it sets all references to that actor to None from Actor list (NOT Object list, which would be very slow). So any objects that have a reference to some destroyed actor which has been destroyed a while ago will cause a crash when tried to be used.