Closed Bug 390185 Opened 17 years ago Closed 17 years ago

signtool error message wrongly uses the term "database"

Categories

(NSS :: Tools, defect, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: nelson, Assigned: nelson)

References

(Blocks 1 open bug, )

Details

Attachments

(1 file)

In signtool's function 
if the call to 
fails for any reason, signtool outputs the error message 
    "the cert \"%s\" does not exist in the database"
                                    ^^^^^^^^^^^^^^^
This is misleading, because it may not have looked "in the database"
AT ALL.  The phrase "in the database" should be removed.
Correction:
In signtool's function create_pk7, if the call to PK11_FindCertFromNickname
fails for any reason ...

See http://lxr.mozilla.org/security/source/security/nss/cmd/signtool/sign.c#270
Attached patch patch v1Splinter Review
This should do it
Attachment #274528 - Flags: review?
Assignee: nobody → nelson
Priority: -- → P3
Target Milestone: --- → 3.12
Attachment #274528 - Flags: review? → review?(neil.williams)
Comment on attachment 274528 [details] [diff] [review]
patch v1

r+
Attachment #274528 - Flags: review?(neil.williams) → review+
Checking in cmd/signtool/sign.c; new revision: 1.13; previous revision: 1.12
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: