fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-05-07 08:28:05 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

shared: refactor nm_utils_g_slist_strlist_cmp() to avoid dead-code warning from Coverity

Browse source 
Coverity sees that "return 0" cannot be reached. Refactor the code,
to avoid the warning.
This commit is contained in:
Thomas Haller 2019-08-02 09:37:46 +02:00
parent d76df4c139
commit 8fb954b81d
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
shared/nm-glib-aux/nm-shared-utils.c
View file

@ -2834,10 +2834,15 @@ nm_utils_g_slist_find_str (const GSList *list,
int int
nm_utils_g_slist_strlist_cmp (const GSList *a, const GSList *b) nm_utils_g_slist_strlist_cmp (const GSList *a, const GSList *b)
{ {
for (; a && b; a = a->next, b = b->next) while (TRUE) {
if (!a)
return !b ? 0 : -1;
if (!b)
return 1;
NM_CMP_DIRECT_STRCMP0 (a->data, b->data); NM_CMP_DIRECT_STRCMP0 (a->data, b->data);
NM_CMP_SELF (a, b); a = a->next;
return 0; b = b->next;
}
} }
/*****************************************************************************/ /*****************************************************************************/