diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2012-03-07 14:43:43 +0100 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2012-03-14 09:07:38 +0100 |
| commit | 53beb6e394c4ff6325b698481c55d3eb44addfd6 (patch) | |
| tree | 5ef9bfc5bb144ec3e1a808e4f41a3ae96cf3a60d /install | |
| parent | eb309b70526f63e39cdb202165a7d01dc1f3b57f (diff) | |
| download | freeipa.git-53beb6e394c4ff6325b698481c55d3eb44addfd6.tar.gz freeipa.git-53beb6e394c4ff6325b698481c55d3eb44addfd6.tar.xz freeipa.git-53beb6e394c4ff6325b698481c55d3eb44addfd6.zip | |
Fixed mask validation in network_validator
Network validator allowed invalid mask format:
* leading zeros: 192.168.0.1/0024
* trailing chars: 192.168.0.1/24abcd
It was fixed.
https://fedorahosted.org/freeipa/ticket/2493
Diffstat (limited to 'install')
| -rw-r--r-- | install/ui/dns.js | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/install/ui/dns.js b/install/ui/dns.js index 4cc6d4c5..512d2cc2 100644 --- a/install/ui/dns.js +++ b/install/ui/dns.js @@ -2231,7 +2231,7 @@ IPA.network_validator = function(spec) { return that.true_result(); } - var address_part, mask_part; + var address_part, mask; if (value.indexOf('/') > -1) { @@ -2239,9 +2239,9 @@ IPA.network_validator = function(spec) { if (parts.length === 2) { address_part = parts[0]; - mask_part = parts[1]; + mask = parts[1]; - if (mask_part === '') return that.false_result(); + if (mask === '') return that.false_result(); } else { return that.false_result(); @@ -2260,14 +2260,12 @@ IPA.network_validator = function(spec) { var address = NET.ip_address(address_part); if (!address.valid) return that.false_result(); - if (mask_part) { - - var mask = parseInt(mask_part, 10); + if (mask) { var mask_length = 32; if (address.type === 'v6') mask_length = 128; - if (isNaN(mask) || mask < 8 || mask > mask_length) { + if (!mask.match(/^[1-9]\d*$/) || mask < 8 || mask > mask_length) { return that.false_result(); } } |