Closed
Bug 1083822
Opened 10 years ago
Closed 10 years ago
Password Handling in SMS-Handler has two different maximum password length limits
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wmathanaraj, Assigned: marta)
References
Details
Attachments
(1 file)
|
484 bytes,
patch
|
freddy
:
review+
|
Details | Diff | Splinter Review |
Impact: Passwords may not be as strong as they could be Fix: Decide on one limit and use it throughout the code (privacy-panel.js:256, and remote_privacy_protection.js:49) Additional note: Dynamically creating the regular expression with the RegExp constructor could allow removing double-checking for disabled commands
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8507717 [details] [diff] [review] bug_1083822.patch Review of attachment 8507717 [details] [diff] [review]: ----------------------------------------------------------------- Yep
Attachment #8507717 -
Flags: review?(fbraun) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•