From 05816dd33396f92b7d9348a2acd18537ac18fa1e Mon Sep 17 00:00:00 2001 From: dequis Date: Sun, 22 Feb 2015 19:04:08 -0300 Subject: coverity: Fix some (harmless?) use-after-free with g_slist_remove() These were passing a pointer to a variable right after it was g_free()'d They are most likely harmless as g_slist_remove() probably just needs the pointer location, but fixing it anyway. --- lib/oauth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/oauth.c') diff --git a/lib/oauth.c b/lib/oauth.c index f15a7135..ac5bc654 100644 --- a/lib/oauth.c +++ b/lib/oauth.c @@ -106,8 +106,8 @@ void oauth_params_del(GSList **params, const char *key) if (strncmp((char *) l->data, key, key_len) == 0 && ((char *) l->data)[key_len] == '=') { - g_free(l->data); *params = g_slist_remove(*params, l->data); + g_free(l->data); } } } -- cgit v1.2.3