This is a code inspection bug. QA: To verify that this bug is fixed, make sure that all callers in of PL_DHashTableInit in the indicated class/file check and handle a failure return result. To verify that this bug is invalid, simply verify (e.g. with lxr) that PL_DHashTableInit is no longer used by the indicated class/file. PL_DHashTableInit returns a PRBool indicating whether it succeeded. PL_DHashTableInit *can* fail. Code can not assume that table->ops or table- >data will be null (in fact it probably will not be), although in all likelyhood table->entryStore will be null it probably isn't safe to assume this. Code at time of bug filing: 376 if (mTransferringRequests.ops) { 377 PL_DHashTableFinish(&mTransferringRequests); 378 mTransferringRequests.ops = nsnull; 379 } 380 PL_DHashTableInit(&mTransferringRequests, &gMapOps, nsnull, 381 sizeof(RequestHashEntry), 16); 382 }

Mass reassign ssaux bugs to nobody

Init is infallible now.