diff options
| author | Martin Basti <mbasti@redhat.com> | 2016-09-02 17:39:01 +0200 |
|---|---|---|
| committer | David Kupka <dkupka@redhat.com> | 2016-09-07 16:22:03 +0200 |
| commit | b232ad463cf43596cdf397e51469df13a89e83fa (patch) | |
| tree | ae9e45f450a29f9640babfb1152f17b6f61625be /client | |
| parent | f3d379071a9af50e38fcc07491bc68b8d3d172a4 (diff) | |
| download | freeipa-b232ad463cf43596cdf397e51469df13a89e83fa.tar.gz freeipa-b232ad463cf43596cdf397e51469df13a89e83fa.tar.xz freeipa-b232ad463cf43596cdf397e51469df13a89e83fa.zip | |
Show warning when net/broadcast IP address is used in installer
https://fedorahosted.org/freeipa/ticket/5814
Reviewed-By: David Kupka <dkupka@redhat.com>
Diffstat (limited to 'client')
| -rwxr-xr-x | client/ipa-client-install | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/client/ipa-client-install b/client/ipa-client-install index 4a263b3d0..6330f1d28 100755 --- a/client/ipa-client-install +++ b/client/ipa-client-install @@ -1651,6 +1651,20 @@ def update_dns(server, hostname, options): root_logger.info("Failed to determine this machine's ip address(es).") return + for ip in update_ips: + if ip.is_network_addr(): + root_logger.warning("IP address %s might be network address", ip) + # fixme: once when loggers will be fixed, we can remove this print + print( + "WARNING: IP address {} might be network address".format(ip), + file=sys.stderr) + if ip.is_broadcast_addr(): + root_logger.warning("IP address %s might be broadcast address", ip) + # fixme: once when loggers will be fixed, we can remove this print + print( + "WARNING: IP address {} might be broadcast address".format(ip), + file=sys.stderr) + update_txt = "debug\n" update_txt += ipautil.template_str(DELETE_TEMPLATE_A, dict(HOSTNAME=hostname)) |