pidgin/pidgin

Fix TALOS-CAN-0141
release-2.x.y
2016-06-02, Andrew Victor
e6159ad42c4c
Parents 5e5e84e8a798
Children 2551d02deb50
Fix TALOS-CAN-0141
2 files changed, 20 insertions(+), 6 deletions(-)
  • +20 -2
    libpurple/protocols/mxit/protocol.c
  • +0 -4
    libpurple/protocols/mxit/roster.c
    • --- a/libpurple/protocols/mxit/protocol.c Sun May 15 05:41:39 2016 -0300
    +++ b/libpurple/protocols/mxit/protocol.c Thu Jun 02 12:49:50 2016 -0500
    @@ -1691,6 +1691,24 @@
    /*------------------------------------------------------------------------
    + * Parse the received mood value, and ensure that it is supported.
    + *
    + * @param value The received mood value.
    + * @return A valid mood value.
    + */
    +static short mxit_parse_mood( const char* value )
    +{
    + short mood = atoi( value );
    +
    + /* ensure that the mood value is valid */
    + if ( ( mood >= MXIT_MOOD_NONE ) && ( mood <= MXIT_MOOD_STRESSED ) )
    + return mood;
    +
    + return MXIT_MOOD_NONE;
    +}
    +
    +
    +/*------------------------------------------------------------------------
    * Process a received contact update packet.
    *
    * @param session The MXit session object
    @@ -1723,7 +1741,7 @@
    contact->presence = mxit_parse_presence( rec->fields[3]->data );
    contact->type = atoi( rec->fields[4]->data );
    - contact->mood = atoi( rec->fields[5]->data );
    + contact->mood = mxit_parse_mood( rec->fields[5]->data );
    if ( rec->fcount > 6 ) {
    /* added in protocol 5.9 - flags & subtype */
    @@ -1780,7 +1798,7 @@
    if ( rec->fcount >= 7 ) /* flags field is included */
    flags = atoi( rec->fields[6]->data );
    - mxit_update_buddy_presence( session, rec->fields[0]->data, mxit_parse_presence( rec->fields[1]->data ), atoi( rec->fields[2]->data ),
    + mxit_update_buddy_presence( session, rec->fields[0]->data, mxit_parse_presence( rec->fields[1]->data ), mxit_parse_mood( rec->fields[2]->data ),
    rec->fields[3]->data, rec->fields[4]->data, flags );
    mxit_update_buddy_avatar( session, rec->fields[0]->data, rec->fields[5]->data );
    }
    --- a/libpurple/protocols/mxit/roster.c Sun May 15 05:41:39 2016 -0300
    +++ b/libpurple/protocols/mxit/roster.c Thu Jun 02 12:49:50 2016 -0500
    @@ -473,10 +473,6 @@
    contact->mood = mood;
    contact->capabilities = flags;
    - /* validate mood */
    - if ( ( contact->mood < MXIT_MOOD_NONE ) || ( contact->mood > MXIT_MOOD_STRESSED ) )
    - contact->mood = MXIT_MOOD_NONE;
    -
    g_strlcpy( contact->customMood, customMood, sizeof( contact->customMood ) );
    // TODO: Download custom mood frame.