405481 - nsISerializable missing from nsNSSCertificate's QI impl

Status: RESOLVED FIXED

(Core :: Security: PSM, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: patch

see bug 388128 comment 21

Comment 1

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 290260 [details] [diff] [review]
patch

Add nsISerializable too, and r+sr=bzbarsky

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: final patch

Fixes an oversight from bug 388128 and is needed for the code there and in bug 369566 to actually work. low risk.

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

(and will also be needed for bug 262116)

Comment 4

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 290266 [details] [diff] [review]
final patch

a=beltzner for drivers

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in security/manager/ssl/src/nsNSSCertificate.cpp;
/cvsroot/mozilla/security/manager/ssl/src/nsNSSCertificate.cpp,v  <--  nsNSSCertificate.cpp
new revision: 1.135; previous revision: 1.134
done