diff options
| author | Thomas Haller <thaller@redhat.com> | 2019-08-02 08:44:12 +0200 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2019-08-02 11:10:50 +0200 |
| commit | 96c999e085588503435620c1242247f2f824e44d (patch) | |
| tree | ddb4989649fe76d4bf2ca3bdd39ee4428bbf8790 /clients/common/nm-meta-setting-desc.c | |
| parent | 650c0cd622fb63c5b64214bd3f7c84c0dbef8b8b (diff) | |
| download | NetworkManager-96c999e085588503435620c1242247f2f824e44d.tar.gz | |
cli: fix type for loop variable in _get_fcn_vlan_xgress_priority_map()
Coverity correctly points out that nm_setting_vlan_get_num_priorities() can return
a negative value (-1 on assertion). Handle that by using the right integer type.
(cherry picked from commit af4a41cc4c1cf17bf9d04477cb6c2ebed5daa6ee)
Diffstat (limited to 'clients/common/nm-meta-setting-desc.c')
| -rw-r--r-- | clients/common/nm-meta-setting-desc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clients/common/nm-meta-setting-desc.c b/clients/common/nm-meta-setting-desc.c index c1a5c2a7fe..5c71868d06 100644 --- a/clients/common/nm-meta-setting-desc.c +++ b/clients/common/nm-meta-setting-desc.c @@ -3755,7 +3755,7 @@ _get_fcn_vlan_xgress_priority_map (ARGS_GET_FCN) NMVlanPriorityMap map_type = _vlan_priority_map_type_from_property_info (property_info); NMSettingVlan *s_vlan = NM_SETTING_VLAN (setting); GString *str = NULL; - guint32 i, num; + gint32 i, num; RETURN_UNSUPPORTED_GET_TYPE (); |