Open Bug 293532 Opened 19 years ago Updated 1 year ago

crlutil should not asserts when used with -i option and -n flag is not present

Categories

(NSS :: Tools, defect, P2)

3.10
x86
All

Tracking

(Not tracked)

People

(Reporter: alvolkov.bgs, Unassigned)

Details

crlutil should extract cert nick name from modified crl.
-n flag should not be a requirement when crltuil starts with -i flag.

Currently crltuil asserts with the following stack with the parameters:
 -d . -M -i <file> -a -o <file>

#0  0xffffe002 in ?? ()
#1  0x42028a73 in abort () from /lib/tls/libc.so.6
#2  0x401322fd in PR_Assert (s=0x805c1c0 "arena != NULL && certHandle != NULL &&
certNickName != NULL", file=0x805c0cf "crlutil.c", ln=0x13b) at
../../../../pr/src/io/prlog.c:538
#3  0x08050926 in DuplicateModCrl (arena=0x8099b80, certHandle=0x808ab10,
cert=0xbfffed20, certNickName=0x0, inFile=0x8068488, decodeOptions=0x0,
importOptions=0x0) at crlutil.c:315
#4  0x08051262 in GenerateCRL (certHandle=0x808ab10, certNickName=0x0,
inCrlInitFile=0x8067eb0, inFile=0x8068488, outFileName=0x80684c8 "root_b64.crl",
ascii=0x1, slotName=0x0, importOptions=0x0, alg=0x0, quiet=0x0,
decodeOptions=0x0, url=0x0, pwdata=0xbfffed88, modifyFlag=0x1) at crlutil.c:627
#5  0x08052042 in main (argc=0x9, argv=0xbfffee64) at crlutil.c:988
#6  0x42015574 in __libc_start_main () from /lib/tls/libc.so.6
Technically, crlutil would extract the issuer subject from the CRL in this case,
not the cert nickname . It would then figure out the right signing cert to use .
QA Contact: bishakhabanerjee → jason.m.reid
Assignee: wtchang → alexei.volkov.bugs
QA Contact: jason.m.reid → tools
Priority: -- → P2
Target Milestone: --- → 3.12
Unsetting target milestone in unresolved bugs whose targets have passed.
Target Milestone: 3.12 → ---

The bug assignee is inactive on Bugzilla, and this bug has priority 'P2'.
:beurdouche, could you have a look please?

For more information, please visit auto_nag documentation.

Assignee: alvolkov.bgs → nobody
Flags: needinfo?(bbeurdouche)
Severity: normal → S3

We have modified the bot to only consider P1 as high priority, so I'm cancelling the needinfo here.

Flags: needinfo?(bbeurdouche)
You need to log in before you can comment on or make changes to this bug.