Open
Bug 1077954
Opened 10 years ago
Updated 2 years ago
the "Add Security Exception" dialog is not robust against unexpected input
Categories
(Core :: Security: PSM, defect, P3)
Tracking
()
REOPENED
People
(Reporter: yfdyh000, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [psm-backlog])
Attachments
(2 files)
Steps to reproduce: Open an Add Security Exception window, get certificate that filled the status2 and even status3 with a few error. No change the location, get certificate that that filled the status1 with good status only. Actual results: The additional status fields not be cleared, the show like "Valid Certificate" and "Unknown Identity" at the same time. Expected results: Proper cleanup previous results. See the http://hg.mozilla.org/mozilla-central/file/b85c260821ab/security/manager/pki/resources/content/exceptionDialog.js#l288.
![]() |
||
Comment 1•10 years ago
|
||
FWIW, this is an example of how the bug can manifest. In particular, the only header and text block that should be shown is: No Information Available Unable to obtain identification status for this site. (Ignore the exact wording - this is from a test build for Bug 1057035)
Blocks: 1029832
Comment 2•9 years ago
|
||
I would like to work on this bug can you please assign this one to me and say where to start and who is gona be the mentor for this bug ?
Flags: needinfo?(yfdyh000)
You're welcome. I'm not sure it should have a mentor, but I'll try to help you, and you can refer to https://developer.mozilla.org/en-US/docs/Introduction.
Assignee: nobody → dhanvicse
Status: NEW → ASSIGNED
Flags: needinfo?(yfdyh000)
Tummala, are you still interested in working on this?
Flags: needinfo?(dhanvicse)
Comment 5•8 years ago
|
||
(In reply to David Keeler [:keeler] (use needinfo?) from comment #4) > Tummala, are you still interested in working on this? David I am still interested in working on this bug, it's just that I need some pointers on where to look at
Flags: needinfo?(dhanvicse)
Actually, it looks like this has already been fixed (I can't reproduce the original issue). Sorry for the false-start, Tummala. Feel free to reach out to me if there's another bug you're interested in working on.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
I guess this problem persists, just need a good testcase.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment 8•8 years ago
|
||
Is this is correct re-production of the bug? I didn't fully understand the "get certificate that filled the status2 and even status3 with a few error." part
Flags: needinfo?(yfdyh000)
1. Add "104.154.89.105 example.org" (is expired.badssl.com) to your hosts file. 2. Go to Options > Advanced > Encryption > View Certificates > Servers, click Add Exception button. 3. Get the status for example.org, you will see the status2 field. It will not be cleared, such as when you reload the state, the state of the site to return to normal, etc.
Flags: needinfo?(yfdyh000)
Reporter | ||
Comment 10•8 years ago
|
||
Further, due to internal cache in Firefox, I could't reproduce it returned to normal website/certificate.
(In reply to YF (Yang) from comment #9) I don't understand what you mean by this last part: > It will not be cleared, such as when you reload the state, the state of the > site to return to normal, etc. Could you provide more concrete steps to reproduce the issue? (For what it's worth, I am seeing the behavior you mention in comment 10 - I believe this is because of internal DNS caching. One thing we could do is to bypass the DNS cache in the add certificate exception dialog, but from what you're saying I'm assuming there's another bug here?)
Flags: needinfo?(yfdyh000)
Reporter | ||
Comment 12•8 years ago
|
||
I encountered a fault website before fill this bug. its security certificate changes, but IP does not change. To reproduce this bug under normal environment, I need to deploy a normal and then fault with multiple error SSL host locally, or through the use of hosts file. Yes, looks reproduce it by hosts file needs to bypass the DNS cache. It is a small bug for UI update, but it does exist. I hope you can understand it from the code (link on comment 0), e.g. 'setText("status2Description", "")' when if (!use2), i.e. remove 'if' and set "" for labels. I guess I can do it now, although make a patch have some trouble for me.
Assignee: dhanvicse → yfdyh000
Flags: needinfo?(yfdyh000)
Reporter | ||
Comment 13•8 years ago
|
||
Sorry, I still encounter some problems for the coding. resetDialog do something similar, but more thorough.
Assignee: yfdyh000 → nobody
Has Regression Range: --- → irrelevant
Has STR: --- → no
Keywords: steps-wanted
Whiteboard: [good first bug]
Component: Security: UI → Security: PSM
Keywords: steps-wanted
Priority: -- → P1
Whiteboard: [psm-assigned]
Has Regression Range: irrelevant → ---
Has STR: no → ---
Assignee: nobody → dkeeler
(Back in October I had intended to complete a patch on this but then I stalled since I ran out of energy to write tests. Hopefully sometime soon I'll complete this. In case I don't ever, I'm updating the summary to be more clear about what I was intending to fix. The basic idea is that the dialog will fail in unhelpful ways if given unexpected input (in either the "location" field or if the encountered certificate changes, etc.))
Summary: Not clear additional status fields in add security exception window → the "Add Security Exception" dialog is not robust against unexpected input
Assignee: dkeeler → nobody
Priority: P1 → P3
Whiteboard: [psm-assigned] → [psm-backlog]
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•