Changeset 9fca0657


Ignore:
Timestamp:
2010-03-15T10:26:25Z (10 years ago)
Author:
Wilmer van der Gaast <wilmer@…>
Branches:
master
Children:
449a51d
Parents:
315dd4c
Message:

Deal with stupid (on first sight, at least) Yahoo! server behaviour causing
double free()s on testing ATM. Apparently it's perfectly legal to give buddy
information with an "end buddy" marker but no "begin buddy".

File:
1 edited

Legend:

Unmodified
Added
Removed
  • protocols/yahoo/libyahoo2.c

    r315dd4c r9fca0657  
    13441344                case 301:       /* End buddy */
    13451345                        if (!strcmp(pair->value, "315") && u) {
    1346                                 users = y_list_prepend(users, u);
     1346                                /* Sometimes user info comes in an odd format with no
     1347                                   "begin buddy" but *with* an "end buddy". Don't add
     1348                                   it twice. */
     1349                                if (!y_list_find(users, u))
     1350                                        users = y_list_prepend(users, u);
    13471351                                u = yd->half_user = NULL;
    13481352                        }
Note: See TracChangeset for help on using the changeset viewer.