447790 - memory leak in prerrortable (PRErrorTableList)

Status: NEW

(NSPR :: NSPR, defect)

Description

[Rick Vestal]

User-Agent:       Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.0; SLCC1; .NET CLR 2.0.50727; .NET CLR 3.0.04506; .NET CLR 3.5.21022)
Build Identifier: cvs source from 20080624

In PR_ErrorInstallTable a table list is contructed, but there is no destruction of this list.

I've added a call:

PR_IMPLEMENT(void)
_PR_CleanupErrorTables()
{
	while ( Table_List != NULL) {
		struct PRErrorTableList *next = Table_List->next;		
		PR_Free( Table_List );
		Table_List = next;
	}	
}

...which is called from PR_Cleanup(), but have only verified it on windows.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Nelson Bolyard (seldom reads bugmail)]

Please attach a patch (the output of diff -pu or cvs diff -pu) showing
your proposed changes.  Thanks.

Comment 2

[hduston]

Patch proposed in bug #557922

https://bugzilla.mozilla.org/show_bug.cgi?id=557922

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.