Correct success returns for fr_lst_{insert, extract}() (#4187)

This change makes them consistent with the heap functions,
simplifying migration from heap to LST.

diff --git a/src/lib/util/lst.c b/src/lib/util/lst.c
index 5b1fd8223f771fa9e461cc7d7c313d680ed9d513..e87cd073982fc2e12c31f1840b31e7755596d35a 100644 (file)
--- a/src/lib/util/lst.c
+++ b/src/lib/util/lst.c
@@ -620,7 +620,7 @@ int fr_lst_extract(fr_lst_t *lst, void *data)
        }
 
        _fr_lst_extract(lst, 0, data);
-       return 1;
+       return 0;
 }
 
 int fr_lst_insert(fr_lst_t *lst, void *data)
@@ -643,7 +643,7 @@ int fr_lst_insert(fr_lst_t *lst, void *data)
        }
 
        _fr_lst_insert(lst, 0, data);
-       return 1;
+       return 0;
 }
 
 fr_lst_index_t fr_lst_num_elements(fr_lst_t *lst)