580137 - Merge SOFTOKEN_3_13_BRANCH onto the NSS trunk

Status: RESOLVED FIXED

(NSS :: Libraries, defect, P1)

Description

[Wan-Teh Chang]

Now that we have created the NSS_3_12_BRANCH, we need to merge the
SOFTOKEN_3_13_BRANCH onto the NSS trunk, and stop using the
SOFTOKEN_3_13_BRANCH.

The attached patch was generated by
cvs -q update -kk -j SOFTOKEN_3_13_BRANCH mozilla/security/nss/lib/freebl
cvs -q update -kk -j SOFTOKEN_3_13_BRANCH mozilla/security/nss/lib/softoken

There have been no changes to mozilla/dbm and mozilla/security/dbm on the
SOFTOKEN_3_13_BRANCH.

Comment 1

[Wan-Teh Chang]

Attachment: Patch

Comment 2

[Wan-Teh Chang]

Attachment: CVS commit log of patch

I checked in the patch on the NSS trunk (NSS 3.13).

Comment 3

[Wan-Teh Chang]

One regression from the merge: new memory leaks on the NSS tinderbox
named "trunk memleak aquash SunOS/i386 32bit", in both DBG and OPT
builds.

In OPT builds, the new memory leak stacks all end in
NSC_GetAttributeValue/sftkdb_GetAttributeValue/sdb_GetAttributeValue/sqlite3_mprintf/sqlite3MemMalloc

In DBG builds, the new memory leak stacks are different, but very similar:
NSC_GetAttributeValue/sftkdb_GetAttributeValue/sdb_GetAttributeValue/sdb_GetAttributeValueNoLock/sqlite3_mprintf/sqlite3_vmprintf/sqlite3StrAccumFinish/sqlite3DbMallocRaw/sqlite3Malloc/mallocWithAlarm/sqlite3MemMalloc

Comment 4

[Wan-Teh Chang]

The memory leak is introduced by the patch checked in
in bug 526231 comment 15.  I will follow up there.

Comment 5

[Wan-Teh Chang]

Attachment: Do not use NO_CHECK_FORK

In NSS 3.13, we should not use NO_CHECK_FORK.
This MXR query shows the remaining occurrences
of NO_CHECK_FORK on the NSS trunk, which this
patch fixes:
http://mxr.mozilla.org/security/search?string=NO_CHECK_FORK

Comment 6

[Christophe Ravel]

Comment on attachment 458857 [details] [diff] [review]
Do not use NO_CHECK_FORK

r=christophe

Comment 7

[Wan-Teh Chang]

Comment on attachment 458857 [details] [diff] [review]
Do not use NO_CHECK_FORK

I checked in this patch on the NSS trunk (NSS 3.13).

Checking in coreconf/Linux2.4.mk;
/cvsroot/mozilla/security/coreconf/Linux2.4.mk,v  <--  Linux2.4.mk
new revision: 1.10; previous revision: 1.9
done
Checking in nss/lib/freebl/blapii.h;
/cvsroot/mozilla/security/nss/lib/freebl/blapii.h,v  <--  blapii.h
new revision: 1.3; previous revision: 1.2
done