Correctness fix in getCaCertForEntityCert (Certificate Manager)

RESOLVED FIXED in mozilla7

Status

()

Core
Security: PSM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: kaie, Assigned: kaie)

Tracking

Trunk
mozilla7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 538488 [details] [diff] [review]
Patch v1

This is a subset of the patch from bug 260364, and has my r=kaie
Assignee: nobody → kaie
Attachment #538488 - Flags: review+
(Assignee)

Updated

6 years ago
Attachment #538488 - Flags: approval2.0?
Attachment #538488 - Flags: approval1.9.2.19?
Attachment #538488 - Flags: approval-mozilla-beta?
Attachment #538488 - Flags: approval-mozilla-aurora?

Comment 2

6 years ago
Fixed on trunk with http://hg.mozilla.org/mozilla-central/rev/b7502e9fbd30
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7

Comment 3

6 years ago
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-
(Assignee)

Comment 4

6 years ago
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.