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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: dietrich, Unassigned)

References

Details

log error:

Assertion failure: padlen > 0 && padlen <= padsize, at p7local.c:588
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);
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?
Yeah, we should.

In the meantime, we can comment out that check from the weave test.
Blocks: 433992
Target Milestone: -- → 0.2
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
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).
Depends on: 401928, 436428
OS: Mac OS X → All
Hardware: PC → All
Summary: crash in test_pbe.js when using trunk sdk → fatal assert in test_crypto_*.js when using trunk sdk
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.
No longer blocks: 433992
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
Component: Weave → General
Product: Mozilla Labs → Weave
QA Contact: weave → general
You need to log in before you can comment on or make changes to this bug.