Closed Bug 1622555 Opened 4 years ago Closed 4 years ago

lowhashtest: broken argument parsing

Categories

(NSS :: Test, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hpj, Assigned: hpj)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:71.0) Gecko/20100101 Firefox/71.0

Steps to reproduce:

Ran all tests in FIPS mode.

Actual results:

lowhashtest reported that all hash functions failed. This happened because it was being called once per hash function with the name of the function to test as its first argument.

Due to bugs in the argument parsing, it would test every function regardless of arguments and report a failure if any one of them failed, resulting in disabled functions (MD5...) erroneously disqualifying runs that were only supposed to test a different function.

Expected results:

If a hash function is specified on the command line, that hash function should be tested instead of all of them. In FIPS mode that means only disabled hash functions would fail instead of all of them.

Attachment #9133395 - Flags: review?(kjacobs.bugzilla)
Assignee: nobody → hpj
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P1
Comment on attachment 9133395 [details] [diff] [review]
nss-lowhashtest-fixes.patch

Thank you!
Attachment #9133395 - Flags: review?(kjacobs.bugzilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.52
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: