diff options
| author | Thomas Haller <thaller@redhat.com> | 2016-03-04 08:58:25 +0100 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2016-03-04 08:59:36 +0100 |
| commit | 9f164989380109a69209d0683b56bf422b33ad11 (patch) | |
| tree | 395966ead3fd6490403b0af954604a6a25a762ff | |
| parent | f2165ab959dc2a66c6c9a820017ec482655a28f5 (diff) | |
| download | NetworkManager-9f164989380109a69209d0683b56bf422b33ad11.tar.gz | |
route-manager/tests: add assertion for adding route
Not checking the return value is marked by coverity. Just workaround it.
| -rw-r--r-- | src/tests/test-route-manager.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/src/tests/test-route-manager.c b/src/tests/test-route-manager.c index 5d60d08c73..b32775da6c 100644 --- a/src/tests/test-route-manager.c +++ b/src/tests/test-route-manager.c @@ -76,15 +76,16 @@ setup_dev1_ip4 (int ifindex) /* Add some route outside of route manager. The route manager * should get rid of it upon sync. */ - nm_platform_ip4_route_add (NM_PLATFORM_GET, - route.ifindex, - NM_IP_CONFIG_SOURCE_USER, - nmtst_inet4_from_string ("9.0.0.0"), - 8, - INADDR_ANY, - 0, - 10, - route.mss); + if (!nm_platform_ip4_route_add (NM_PLATFORM_GET, + route.ifindex, + NM_IP_CONFIG_SOURCE_USER, + nmtst_inet4_from_string ("9.0.0.0"), + 8, + INADDR_ANY, + 0, + 10, + route.mss)) + g_assert_not_reached (); route.source = NM_IP_CONFIG_SOURCE_USER; inet_pton (AF_INET, "6.6.6.0", &route.network); @@ -428,9 +429,8 @@ setup_dev1_ip6 (int ifindex) 48, in6addr_any, 10, - 0)) { + 0)) g_assert_not_reached (); - } route = nmtst_platform_ip6_route_full ("2001:db8:8086::", 48, |