diff options
| author | dequis <dx@dxzone.com.ar> | 2015-02-22 19:04:08 -0300 |
|---|---|---|
| committer | dequis <dx@dxzone.com.ar> | 2015-02-22 19:44:40 -0300 |
| commit | 05816dd33396f92b7d9348a2acd18537ac18fa1e (patch) | |
| tree | 980ad6ad4c5efc040d19ec09ed1cecc0104b17eb /lib/oauth.c | |
| parent | 91f06e5535d3711da06533495b197c1b8e545f82 (diff) | |
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.
Diffstat (limited to 'lib/oauth.c')
| -rw-r--r-- | lib/oauth.c | 2 |
1 files changed, 1 insertions, 1 deletions
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); } } } |