NSS 3.11.1 libnssckbi.so built with Forte 6 update 2 doesn't have version strings.

RESOLVED FIXED in 3.11.2

Status

NSS
Libraries
--
minor
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Wan-Teh Chang, Unassigned)

Tracking

3.11.1
3.11.2
Sun
Solaris

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

974 bytes, patch
Alexei Volkov
: review+
Nelson Bolyard (seldom reads bugmail)
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

12 years ago
I found that our NSS_3_11_1_RTM libnssckbi.so built with
Forte 6 update 2 on Solaris 9 SPARC (32-bit and 64-bit,
debug and optimized) doesn't have the 'what' and 'ident'
version strings.

Christophe, does your libnssckbi.so have the same problem?
Please try these commands:

  what libnssckbi.so | grep NSS
  ident libnssckbi.so | grep NSS
(Reporter)

Comment 1

12 years ago
I just built NSS_3_11_1_RTM on Solaris 9 SPARC with
Sun Studio 11.  libnssckbi.so in debug builds (32
and 64 bits) has the version string, but libnssckbi.so
in optimized builds (32 and 64 bits) doesn't have the
version string.

Comment 2

12 years ago
We build on Sun Studio 11.

OPT/32:
$ what libnssckbi.so | grep NSS
$ ident libnssckbi.so | grep NSS
ident warning: no id keywords in libnssckbi.so

DBG/32:
$ what libnssckbi.so | grep NSS
$ ident libnssckbi.so | grep NSS

OPT/64:
$ what libnssckbi.so | grep NSS
$ ident libnssckbi.so | grep NSS
ident warning: no id keywords in libnssckbi.so

DBG/64:
$ what libnssckbi.so | grep NSS
$ ident libnssckbi.so | grep NSS
(In reply to comment #2)
> We build on Sun Studio 11.

So, according to your output, we get a warning on optimized builds,
and NO OUTPUT AT ALL on debug builds?  
The expression "not ready for prime time" comes to mind.
(Reporter)

Comment 4

12 years ago
Created attachment 225290 [details] [diff] [review]
Proposed patch

We need to add a dummy reference to the __nss_builtins_rcsid
and __nss_builtins_sccsid strings.  Otherwise, the compiler
may optimize away these two version strings.  Please see
lib/ssl/sslcon.c and lib/ssl/sslver.c for an example of this
dummy reference.  (Search for "rcsid" in those two files.)
Attachment #225290 - Flags: superreview?(nelson)
Attachment #225290 - Flags: review?(christophe.ravel.bugs)
(Reporter)

Comment 5

12 years ago
Nelson, many NSS source files have RCS id strings but they
are ifdef'd with the DEBUG macro.  This is why the RCS 'ident'
command only warns about the lack of RCS id keywords in our
optimized builds.

Comment 6

12 years ago
Comment on attachment 225290 [details] [diff] [review]
Proposed patch

I have to pass the review to somebody else (my knowledge in C is too far away now).
Alexei, do you want to take the review ?
Attachment #225290 - Flags: review?(christophe.ravel.bugs) → review?(alexei.volkov)
Comment on attachment 225290 [details] [diff] [review]
Proposed patch

looks just like the way we do it in other shared libs.
r=nelson
Attachment #225290 - Flags: superreview?(nelson) → superreview+
(Reporter)

Comment 8

12 years ago
I checked in the proposed patch on the NSS trunk (3.12)
and NSS_3_11_BRANCH (3.11.2).
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.11.2

Comment 9

11 years ago
Comment on attachment 225290 [details] [diff] [review]
Proposed patch

I didn't see this request until recently because review request was sent to alexei.volkov@sun.com account. The account is now deleted by bugzilla admin.
Attachment #225290 - Flags: review?(alexei.volkov.bugs) → review+
You need to log in before you can comment on or make changes to this bug.