[#VWR-6620] Crash at shutdown due to newview/llfloaterproperties.cpp adding a singleton observer to gInventory

1. Second Life Viewer - VWR

Crash at shutdown due to newview/llfloaterproperties.cpp adding a singleton observer to gInventory

Created: 16/Apr/08 03:18 PM Updated: 07/May/08 05:07 PM

Return to search

Component/s:

Source Code

Affects Version/s:

Source code

Fix Version/s:

1.20 Release Candidate

Environment:

« Hide

Second Life 1.20.2 (0) Apr 16 2008 19:21:07 (Second Life Release)

You are at 238576.9, 244744.8, 25.5 in Hippotropolis located at sim5714.agni.lindenlab.com (8.2.35.16:13004)
Second Life Server 1.20.0.83892

CPU: Intel(R) Core(TM)2 Duo CPU T7100 @ 1.80GHz
Memory: 2012 MB
OS Version: Linux 2.6.22-carjay #3 SMP Thu Dec 20 14:22:13 CET 2007 x86_64
Graphics Card Vendor: NVIDIA Corporation
Graphics Card: GeForce 8400M G/PCI/SSE2
OpenGL Version: 2.1.1 NVIDIA 100.14.19
LLMozLib Version: [LLMediaImplLLMozLib] - 2.01.14497 (Mozilla GRE version 1.8.1.11_0000000000)
Packets Lost: 0/268 (0.0%)

Issue Links:

Duplicate

This issue duplicates:

VWR-6389 SmartHeap Library error msgs immediately after log-in or on exit--viewer sometimes disappears from Windows, without error msg, immediately after log-in

Description

« Hide

Against the current public viewer 1.20.2(0) branch (r479)

The constructor of LLFloaterProperties (in llfloaterproperties.cpp) adds an LLPropertiesObserver instance to the global inventory model gInventory.

Problem is that this instance has recently been made a singleton, so it's actually static.

At shutdown the global destructor of gInventory deletes all observers that are still in its list which of course is a bad idea for a singleton instance.

All

Comments

Change History

Sort Order:

Carjay McGinnis made changes - 16/Apr/08 03:19 PM

Field	Original Value	New Value
Summary	Crash at shutdown due to lfloaterproperties adding a singleton observer to gInventory	Crash at shutdown due to llfloaterproperties adding a singleton observer to gInventory

Carjay McGinnis made changes - 16/Apr/08 03:20 PM

Summary

Crash at shutdown due to llfloaterproperties adding a singleton observer to gInventory

Crash at shutdown due to newview/llfloaterproperties.cpp adding a singleton observer to gInventory

Carjay McGinnis made changes - 16/Apr/08 03:33 PM

Description

Against the current public viewer 1.20.2(0) branch (r479)

The constructor of LLFloaterProperties (in llfloaterproperties.cpp) adds an LLPropertiesObserver instance to the global inventory model gInventory.

Problem is that this instance has recently been made a Singleton, so it's actually static.

At shutdown the global destructor of gInventory deletes all observers that are still in its list which of course is a bad idea for a singleton instance.

Against the current public viewer 1.20.2(0) branch (r479)

The constructor of LLFloaterProperties (in llfloaterproperties.cpp) adds an LLPropertiesObserver instance to the global inventory model gInventory.

Problem is that this instance has recently been made a singleton, so it's actually static.

At shutdown the global destructor of gInventory deletes all observers that are still in its list which of course is a bad idea for a singleton instance.

Alexa Linden made changes - 17/Apr/08 02:00 PM

Resolution		Needs More Info [ 4 ]
Status	Open [ 1 ]	Resolved [ 5 ]

Carjay McGinnis made changes - 17/Apr/08 04:22 PM

Environment

Linux Kubuntu Feisty x86_64

Soft Linden made changes - 18/Apr/08 05:36 AM

Assignee		Soft Linden [ Soft Linden ]
Status	Resolved [ 5 ]	Reopened [ 4 ]
Resolution	Needs More Info [ 4 ]

Carjay McGinnis made changes - 07/May/08 04:59 PM

Link

This issue duplicates VWR-6389 [ VWR-6389 ]

Carjay McGinnis made changes - 07/May/08 05:07 PM

Status	Reopened [ 4 ]	Closed [ 6 ]
Fix Version/s		1.20 Release Candidate [ 10300 ]
Resolution		Fixed [ 1 ]

Sue Linden made changes - 13/Nov/08 11:50 AM

Workflow

jira-2007-12-22a [ 54949 ]

jira-2008-11-14 [ 77922 ]

Sue Linden made changes - 13/Nov/08 06:57 PM

Workflow

jira-2008-11-14 [ 77922 ]

jira-2008-11-14a [ 140781 ]