478931 - object leak in pkix_List_MergeLists function

Status: RESOLVED FIXED

(NSS :: Libraries, defect)

Description

[Alexei Volkov]

Attachment: pkix_list.c patch v1 - fix code that leaks object referenced by "item"

The leak has been detected by object leak testing tinderbox, but was not reported by the tinderbox website. The reason for that is a failure to upload a log file what is bigger than 50Mb. 
The memory leak checking tinderbox generates a big logs in case of memory leaks
if there is NSPR_LOG_MODULES variable is set to pkix:1 in the environment. 

Slavo will restart the tinderbox script on the machine without NSPR_LOG_MODULES variable. This will make the log to be within the allowed limits.

The code that was causing the leak is in pkix_List_MergeLists. The object referenced by (PKIX_PL_Object*)item was not de-referenced in case of an error.

Comment 1

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 362759 [details] [diff] [review]
pkix_list.c patch v1 - fix code that leaks object referenced by "item"

r=nelson

Comment 2

[Alexei Volkov]

fix integrated