365966 - infinite recursive call in VFY_VerifyDigestDirect

Status: RESOLVED FIXED

(NSS :: Libraries, defect, P1)

Description

[Alexei Volkov]

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

[Robert Relyea]

The inner call should be vfy_VerifyDigest.

bob

Comment 2

[Nelson Bolyard (seldom reads bugmail)]

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.

Comment 3

[Alexei Volkov]

Attachment: changing calling function name

Tests will be integrated as a part of a fix for bug 301496.

Comment 4

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 250902 [details] [diff] [review]
changing calling function name

r=nelson

Comment 5

[Alexei Volkov]

/cvsroot/mozilla/security/nss/lib/cryptohi/secvfy.c,v  <--  secvfy.c
new revision: 1.21; previous revision: 1.20