pidgin/pidgin

facebook: fixed a few issues reported by Coverity

2015-12-28, James Geboski
94b23a04da62
Parents c7a1976029ea
Children 609d20c56162
facebook: fixed a few issues reported by Coverity
2 files changed, 3 insertions(+), 2 deletions(-)
  • +2 -2
    libpurple/protocols/facebook/http.c
  • +1 -0
    libpurple/protocols/facebook/mqtt.c
    • --- a/libpurple/protocols/facebook/http.c Mon Dec 28 16:02:59 2015 -0500
    +++ b/libpurple/protocols/facebook/http.c Mon Dec 28 16:51:11 2015 -0500
    @@ -152,11 +152,11 @@
    if (isurl) {
    data = strchr(data, '?');
    - if (data++ == NULL) {
    + if (data == NULL) {
    return params;
    }
    - tail = strchr(data, '#');
    + tail = strchr(++data, '#');
    if (tail != NULL) {
    data = g_strndup(data, tail - data);
    --- a/libpurple/protocols/facebook/mqtt.c Mon Dec 28 16:02:59 2015 -0500
    +++ b/libpurple/protocols/facebook/mqtt.c Mon Dec 28 16:51:11 2015 -0500
    @@ -469,6 +469,7 @@
    }
    if (!fb_mqtt_message_read_mid(msg, &mid)) {
    + g_free(str);
    break;
    }