Changeset 05816dd for lib


Ignore:
Timestamp:
2015-02-22T22:44:40Z (5 years ago)
Author:
dequis <dx@…>
Branches:
master
Children:
570f183
Parents:
91f06e5
git-author:
dequis <dx@…> (22-02-15 22:04:08)
git-committer:
dequis <dx@…> (22-02-15 22:44:40)
Message:

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.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • lib/oauth.c

    r91f06e5 r05816dd  
    107107                if (strncmp((char *) l->data, key, key_len) == 0 &&
    108108                    ((char *) l->data)[key_len] == '=') {
     109                        *params = g_slist_remove(*params, l->data);
    109110                        g_free(l->data);
    110                         *params = g_slist_remove(*params, l->data);
    111111                }
    112112        }
Note: See TracChangeset for help on using the changeset viewer.