summaryrefslogtreecommitdiff
path: root/libnm-util/nm-setting-ip4-config.c
diff options
context:
space:
mode:
authorDan Williams <dcbw@redhat.com>2011-07-05 19:26:43 -0500
committerDan Williams <dcbw@redhat.com>2011-07-05 19:26:57 -0500
commit39b2d78ee3f197652ee5d6a6773e81372a62bd74 (patch)
treefbf59a101c8ef2dafdfe62b0f2942b385a350dc6 /libnm-util/nm-setting-ip4-config.c
parent24855b02ecd72e499c4c05a3b98db1902bd15608 (diff)
downloadNetworkManager-39b2d78ee3f197652ee5d6a6773e81372a62bd74.tar.gz
libnm-util: enforce IPv4 prefix range validity
Prefixes are between 1 and 32 inclusive so we might as well enforce that.
Diffstat (limited to 'libnm-util/nm-setting-ip4-config.c')
-rw-r--r--libnm-util/nm-setting-ip4-config.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/libnm-util/nm-setting-ip4-config.c b/libnm-util/nm-setting-ip4-config.c
index a632c4212c..6b164ea88a 100644
--- a/libnm-util/nm-setting-ip4-config.c
+++ b/libnm-util/nm-setting-ip4-config.c
@@ -1423,7 +1423,7 @@ nm_ip4_address_get_prefix (NMIP4Address *address)
/**
* nm_ip4_address_set_prefix:
* @address: the #NMIP4Address
- * @prefix: the address prefix, a number between 0 and 32 inclusive
+ * @prefix: the address prefix, a number between 1 and 32 inclusive
*
* Sets the IPv4 address prefix.
**/
@@ -1433,6 +1433,7 @@ nm_ip4_address_set_prefix (NMIP4Address *address, guint32 prefix)
g_return_if_fail (address != NULL);
g_return_if_fail (address->refcount > 0);
g_return_if_fail (prefix <= 32);
+ g_return_if_fail (prefix > 0);
address->prefix = prefix;
}
@@ -1636,7 +1637,7 @@ nm_ip4_route_get_prefix (NMIP4Route *route)
/**
* nm_ip4_route_set_prefix:
* @route: the #NMIP4Route
- * @prefix: the prefix, a number between 0 and 32 inclusive
+ * @prefix: the prefix, a number between 1 and 32 inclusive
*
* Sets the IPv4 prefix of this route.
**/
@@ -1645,6 +1646,8 @@ nm_ip4_route_set_prefix (NMIP4Route *route, guint32 prefix)
{
g_return_if_fail (route != NULL);
g_return_if_fail (route->refcount > 0);
+ g_return_if_fail (prefix <= 32);
+ g_return_if_fail (prefix > 0);
route->prefix = prefix;
}