Closed
Bug 435120
Opened 16 years ago
Closed 16 years ago
fatal assert in test_crypto_*.js when using trunk sdk
Categories
(Cloud Services :: General, defect)
Cloud Services
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
0.3
People
(Reporter: dietrich, Unassigned)
References
Details
log error: Assertion failure: padlen > 0 && padlen <= padsize, at p7local.c:588
Reporter | ||
Comment 1•16 years ago
|
||
crashing line is the decrypt() call below. // The following check with wrong password must cause decryption to fail // beuase of used padding-schema cipher, RFC 3852 Section 6.3 let failure = false; try { pbe.decrypt("wrongpassphrase", cipherTxt);
Reporter | ||
Comment 2•16 years ago
|
||
ok, this is because that check is asserting, and i'm using a debug build. should probably not assert, per the comment here (thanks dan): https://bugzilla.mozilla.org/show_bug.cgi?id=400742#c40 should we keep this open to cover fixing that?
Comment 3•16 years ago
|
||
Yeah, we should. In the meantime, we can comment out that check from the weave test.
Comment 4•16 years ago
|
||
This bug now affects using weave, not just tests (because we actively use our nss component). So we require a non-debug build atm, or you need to comment out the assertion. Pushing to later, we have no real fix for this atm.
Target Milestone: 0.2 → 0.3
Comment 5•16 years ago
|
||
test_pbe.js is gone now. I don't think the original assert mentioned here is actually hit anymore, although it was fixed in bug 436428 anyway. However, I think we do hit the assert in 401928 comment 57. Should be harmless for Weave's purposes (the assert can safely be commented out in your Moz build).
Comment 6•16 years ago
|
||
Ah sorry, I confused the two asserts. The problem is essentially the same, though: debug builds will crash unless they comment out the offending assertion.
Comment 7•16 years ago
|
||
I don't think we plan to do anything about this problem, other than wait for future NSS builds that don't error out. Thus, I'm wontfixing this bug. Reopen if there is anything we should be doing here.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
Updated•15 years ago
|
Component: Weave → General
Product: Mozilla Labs → Weave
Updated•15 years ago
|
QA Contact: weave → general
You need to log in
before you can comment on or make changes to this bug.
Description
•