infinite recursive call in VFY_VerifyDigestDirect

RESOLVED FIXED in 3.12

Status

NSS
Libraries
P1
normal
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Alexei Volkov, Assigned: Alexei Volkov)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1020 bytes, patch
Nelson Bolyard (seldom reads bugmail)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

11 years ago
Integrated as a part of patch for bug 320583.

SECStatus
VFY_VerifyDigestDirect(const SECItem *digest, const SECKEYPublicKey *key, 
		       const SECItem *sig, SECOidTag encAlg, 
		       SECOidTag hashAlg, void *wincx)
{
     return VFY_VerifyDigestDirect(digest, key, sig, encAlg, hashAlg, wincx);       
}

Comment 1

11 years ago
The inner call should be vfy_VerifyDigest.

bob
Alexei, Thank you for testing this new code in NSS 3.12.  
It clearly hasn't previously been tested.
Please code a patch per bob's comment 1, and test that to see if it works.  
Also, please add a comment to this bug explaining what command (test program)
found this bug, and plan to add that to the appropriate NSS test script.
Priority: -- → P1
Summary: infinit recursive call in VFY_VerifyDigestDirect → infinite recursive call in VFY_VerifyDigestDirect
Version: 3.12 → trunk
(Assignee)

Comment 3

11 years ago
Created attachment 250902 [details] [diff] [review]
changing calling function name

Tests will be integrated as a part of a fix for bug 301496.
Assignee: rrelyea → alexei.volkov.bugs
Status: NEW → ASSIGNED
Attachment #250902 - Flags: review?(nelson)
Comment on attachment 250902 [details] [diff] [review]
changing calling function name

r=nelson
Attachment #250902 - Flags: review?(nelson) → review+
(Assignee)

Comment 5

11 years ago
/cvsroot/mozilla/security/nss/lib/cryptohi/secvfy.c,v  <--  secvfy.c
new revision: 1.21; previous revision: 1.20
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Updated

11 years ago
Duplicate of this bug: 339878
You need to log in before you can comment on or make changes to this bug.