Last Comment Bug 390728 - pkix_pl_OcspRequest_Create throws an error if it was not able to get AIA location
: pkix_pl_OcspRequest_Create throws an error if it was not able to get AIA loca...
Status: RESOLVED FIXED
PKIX
:
Product: NSS
Classification: Components
Component: Libraries (show other bugs)
: 3.12
: All All
: P2 normal (vote)
: 3.12
Assigned To: Alexei Volkov
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-08-02 16:15 PDT by Alexei Volkov
Modified: 2007-08-03 10:38 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
simple fix for the problem (1.25 KB, patch)
2007-08-02 16:15 PDT, Alexei Volkov
nelson: review+
Details | Diff | Splinter Review

Description Alexei Volkov 2007-08-02 16:15:16 PDT
Created attachment 275039 [details] [diff] [review]
simple fix for the problem

The reason for the failure was the change http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/security/nss/lib/certhigh/ocsp.c&rev=1.26 made to fix bug 287850.

The fix made CERT_GetOCSPAuthorityInfoAccessLocation set code different from what was expected by libpkix in case when AIA extension is not present or undecodeble.

As the result, an error was thrown and taken as cert revocation reason.
Comment 1 Nelson Bolyard (seldom reads bugmail) 2007-08-03 01:53:20 PDT
Comment on attachment 275039 [details] [diff] [review]
simple fix for the problem

r=nelson for the trunk
Comment 2 Nelson Bolyard (seldom reads bugmail) 2007-08-03 01:56:11 PDT
Of course, this is c code, which doesn't really ever "throw" anything. :)
Comment 3 Alexei Volkov 2007-08-03 10:38:30 PDT
/cvsroot/mozilla/security/nss/lib/libpkix/pkix_pl_nss/pki/pkix_pl_ocsprequest.c,v  <--  pkix_pl_ocsprequest.c
new revision: 1.3; previous revision: 1.2

Note You need to log in before you can comment on or make changes to this bug.