Closed Bug 663362 Opened 14 years ago Closed 14 years ago

Correctness fix in getCaCertForEntityCert (Certificate Manager)

Categories

(Core :: Security: PSM, defect)

Product:
Core
Component:
Security: PSM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla7

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(1 file)

Patch v1
811 bytes, patch
KaiE
: review+
asa
: approval-mozilla-aurora-
asa
: approval-mozilla-beta-
Details | Diff | Splinter Review 
In bug 260364, the patch includes a fix for getCaCertForEntityCert.

On branches that don't pick up 260364, we should add least add this correctness fix.
Attached patch Patch v1 — — Splinter Review 
This is a subset of the patch from bug 260364, and has my r=kaie
Assignee: nobody → kaie
Attachment #538488 - Flags: review+
Attachment #538488 - Flags: approval2.0?
Attachment #538488 - Flags: approval1.9.2.19?
Attachment #538488 - Flags: approval-mozilla-beta?
Attachment #538488 - Flags: approval-mozilla-aurora?
Fixed on trunk with http://hg.mozilla.org/mozilla-central/rev/b7502e9fbd30
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Comment on attachment 538488 [details] [diff] [review]
Patch v1

this is not the kind of change we're interested in for the stabilizing branches.
Attachment #538488 - Flags: approval-mozilla-beta?
Attachment #538488 - Flags: approval-mozilla-beta-
Attachment #538488 - Flags: approval-mozilla-aurora?
Attachment #538488 - Flags: approval-mozilla-aurora-
Comment on attachment 538488 [details] [diff] [review]
Patch v1

If correctness fixes are not desired for soon-to-be-released software, then there is no point to fix them in stable branches. Removing requests.
Attachment #538488 - Flags: approval2.0?
Attachment #538488 - Flags: approval1.9.2.19?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: