diff options
author | Thomas Haller <thaller@redhat.com> | 2019-08-02 17:13:15 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2019-08-02 17:13:16 +0200 |
commit | 1b30797bc16c7e0cc2e6c5686e52aa12ef5a5425 (patch) | |
tree | 29cc675207943cadfac01ed61cb716a23c3cd534 | |
parent | 50ae9c936cd5d0838c68123655feb05c1267e89e (diff) | |
download | NetworkManager-1b30797bc16c7e0cc2e6c5686e52aa12ef5a5425.tar.gz |
core: assert for valid arguments in _addresses_sort_cmp()
Coverity thinks that the arguments could be %NULL. Add an assertion,
hoping to silence coverity.
-rw-r--r-- | src/nm-ip6-config.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/nm-ip6-config.c b/src/nm-ip6-config.c index 1096fb4246..1810d51185 100644 --- a/src/nm-ip6-config.c +++ b/src/nm-ip6-config.c @@ -318,6 +318,9 @@ sort_captured_addresses (const CList *lst_a, const CList *lst_b, gconstpointer u const NMPlatformIP6Address *addr_a = NMP_OBJECT_CAST_IP6_ADDRESS (c_list_entry (lst_a, NMDedupMultiEntry, lst_entries)->obj); const NMPlatformIP6Address *addr_b = NMP_OBJECT_CAST_IP6_ADDRESS (c_list_entry (lst_b, NMDedupMultiEntry, lst_entries)->obj); + nm_assert (addr_a); + nm_assert (addr_b); + return _addresses_sort_cmp (addr_a, addr_b, ((NMSettingIP6ConfigPrivacy) GPOINTER_TO_INT (user_data)) == NM_SETTING_IP6_CONFIG_PRIVACY_PREFER_TEMP_ADDR); } |