Mercurial > pidgin.yaz
changeset 23618:c3506ae277f6
merge of '2a68f533ef12aaf2b26c7827857687c4ff04b229'
and '8022890df03cfa8cce52f7dee5b12ad58b64bbe6'
author | Elliott Sales de Andrade <qulogic@pidgin.im> |
---|---|
date | Sat, 19 Jul 2008 03:10:10 +0000 |
parents | a9db0aec7e59 (diff) 2eefa8dc7481 (current diff) |
children | d70aad03b491 e53a21941d43 |
files | |
diffstat | 22 files changed, 111 insertions(+), 71 deletions(-) [+] |
line wrap: on
line diff
--- a/configure.ac Sat Jul 19 03:07:33 2008 +0000 +++ b/configure.ac Sat Jul 19 03:10:10 2008 +0000 @@ -47,7 +47,7 @@ m4_define([purple_major_version], [2]) m4_define([purple_minor_version], [5]) m4_define([purple_micro_version], [0]) -m4_define([purple_version_suffix], []) +m4_define([purple_version_suffix], [devel]) m4_define([purple_version], [purple_major_version.purple_minor_version.purple_micro_version]) m4_define([purple_display_version], purple_version[]m4_ifdef([purple_version_suffix],[purple_version_suffix])) @@ -56,7 +56,7 @@ m4_define([gnt_major_version], [2]) m4_define([gnt_minor_version], [5]) m4_define([gnt_micro_version], [0]) -m4_define([gnt_version_suffix], []) +m4_define([gnt_version_suffix], [devel]) m4_define([gnt_version], [gnt_major_version.gnt_minor_version.gnt_micro_version]) m4_define([gnt_display_version], gnt_version[]m4_ifdef([gnt_version_suffix],[gnt_version_suffix]))
--- a/libpurple/protocols/bonjour/jabber.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/bonjour/jabber.c Sat Jul 19 03:10:10 2008 +0000 @@ -672,7 +672,6 @@ bonjour_jabber_start(BonjourJabber *jdata) { struct sockaddr_in my_addr; - int yes = 1; int i; gboolean bind_successful; @@ -686,16 +685,6 @@ return -1; } - /* Make the socket reusable */ - if (setsockopt(jdata->socket, SOL_SOCKET, SO_REUSEADDR, &yes, sizeof(int)) != 0) - { - purple_debug_error("bonjour", "Error setting socket options: %s\n", g_strerror(errno)); - purple_connection_error_reason (jdata->account->gc, - PURPLE_CONNECTION_ERROR_NETWORK_ERROR, - _("Error setting socket options")); - return -1; - } - memset(&my_addr, 0, sizeof(struct sockaddr_in)); my_addr.sin_family = AF_INET; @@ -709,6 +698,8 @@ bind_successful = TRUE; break; } + + purple_debug_info("bonjour", "Unable to bind to port %u.(%s)\n", jdata->port, g_strerror(errno)); jdata->port++; }
--- a/libpurple/protocols/oscar/flap_connection.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/oscar/flap_connection.c Sat Jul 19 03:10:10 2008 +0000 @@ -299,12 +299,12 @@ conn->connect_data = NULL; } - if (conn->connect_data != NULL) + if (conn->new_conn_data != NULL) { if (conn->type == SNAC_FAMILY_CHAT) { oscar_chat_destroy(conn->new_conn_data); - conn->connect_data = NULL; + conn->new_conn_data = NULL; } }
--- a/libpurple/protocols/qq/AUTHORS Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/AUTHORS Sat Jul 19 03:10:10 2008 +0000 @@ -16,7 +16,7 @@ Lovely Patch Writers ===== -Gnap.An : message displaying, documentation +gnap : message displaying, documentation manphiz : qun processing moo : qun processing Coly Li : qun processing @@ -32,4 +32,4 @@ rlaager@pidgin.im OpenQ Team LumaQQ Team -OpenQ@GoogleGroup +OpenQ Google Group
--- a/libpurple/protocols/qq/ChangeLog Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/ChangeLog Sat Jul 19 03:10:10 2008 +0000 @@ -1,3 +1,11 @@ +2008.07.12 - ccpaging <ecc_hy(at)hotmail.com> + * Fixed: Always lost connection. Now send keep alive packet in every 30 seconds + * Minor fix for debug information + * Filter \r\n and replace with SPCAE in group notive + * Fixed a memory leak + * Tickets: + * Fixes #4024. + 2008.06.29 - csyfek <csyfek(at)gmail.com> * Minor bug fix * Add some doxygen syntax for preparing development documentation
--- a/libpurple/protocols/qq/buddy_info.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/buddy_info.c Sat Jul 19 03:10:10 2008 +0000 @@ -208,8 +208,8 @@ return FALSE; } - static PurpleNotifyUserInfo * -info_to_notify_user_info(const contact_info *info) +static PurpleNotifyUserInfo * + info_to_notify_user_info(const contact_info *info) { PurpleNotifyUserInfo *user_info = purple_notify_user_info_new(); const gchar *intro; @@ -826,7 +826,8 @@ PurpleBuddy *b; qq_data *qd; qq_buddy *q_bud; - gchar *alias_utf8, *purple_name; + gchar *alias_utf8; + gchar *purple_name; PurpleAccount *account = purple_connection_get_account(gc); qd = (qq_data *) gc->proto_data; @@ -1009,7 +1010,7 @@ bytes += qq_get16(&level, decr_buf + bytes); bytes += qq_get16(&timeRemainder, decr_buf + bytes); purple_debug(PURPLE_DEBUG_INFO, "QQ", - "Level packet entry:\nuid: %d\nonlineTime: %d\nlevel: %d\ntimeRemainder: %d\n", + "Level uid: %d, onlineTime: %d, level: %d, timeRemainder: %d\n", uid, onlineTime, level, timeRemainder); purple_name = uid_to_purple_name(uid); b = purple_find_buddy(account, purple_name);
--- a/libpurple/protocols/qq/buddy_list.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/buddy_list.c Sat Jul 19 03:10:10 2008 +0000 @@ -234,12 +234,11 @@ "qq_process_get_buddies_online_reply: Dangerous error! maybe protocol changed, notify developers!\n"); } - if (position != QQ_FRIENDS_ONLINE_POSITION_END) { - purple_debug(PURPLE_DEBUG_INFO, "QQ", "Received %d online buddies, nextposition=%u\n", - count, (guint) position); - if (position != QQ_FRIENDS_ONLINE_POSITION_START) { - purple_debug(PURPLE_DEBUG_INFO, "QQ", "Requesting for more online buddies\n"); - } + purple_debug(PURPLE_DEBUG_INFO, "QQ", "Received %d online buddies, nextposition=%u\n", + count, (guint) position); + if (position != QQ_FRIENDS_ONLINE_POSITION_END + && position != QQ_FRIENDS_ONLINE_POSITION_START) { + purple_debug(PURPLE_DEBUG_INFO, "QQ", "Requesting for more online buddies\n"); qq_send_packet_get_buddies_online(gc, position); } else { purple_debug(PURPLE_DEBUG_INFO, "QQ", "All online buddies received\n"); @@ -318,8 +317,8 @@ if (QQ_DEBUG) { purple_debug(PURPLE_DEBUG_INFO, "QQ", - "buddy [%09d]: flag1=0x%02x, comm_flag=0x%02x\n", - q_bud->uid, q_bud->flag1, q_bud->comm_flag); + "buddy [%09d]: flag1=0x%02x, comm_flag=0x%02x, nick=%s\n", + q_bud->uid, q_bud->flag1, q_bud->comm_flag, q_bud->nickname); } name = uid_to_purple_name(q_bud->uid); @@ -432,7 +431,9 @@ purple_debug(PURPLE_DEBUG_INFO, "QQ", "Get all list done, %d buddies and %d Quns\n", i, j); purple_debug(PURPLE_DEBUG_INFO, "QQ", "Received %d buddies and %d groups, nextposition=%u\n", i, j, (guint) position); - if (position != QQ_FRIENDS_ALL_LIST_POSITION_START && position != QQ_FRIENDS_ALL_LIST_POSITION_START) { + + if (position != QQ_FRIENDS_ALL_LIST_POSITION_START + && position != QQ_FRIENDS_ALL_LIST_POSITION_END) { purple_debug(PURPLE_DEBUG_INFO, "QQ", "Requesting for more buddies and groups\n"); qq_send_packet_get_all_list_with_group(gc, position); } else {
--- a/libpurple/protocols/qq/buddy_status.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/buddy_status.c Sat Jul 19 03:10:10 2008 +0000 @@ -205,7 +205,9 @@ b = purple_find_buddy(gc->account, name); g_free(name); q_bud = (b == NULL) ? NULL : (qq_buddy *) b->proto_data; - qq_update_buddy_contact(gc, q_bud); + if (q_bud != NULL) { + qq_update_buddy_contact(gc, q_bud); + } } }
--- a/libpurple/protocols/qq/char_conv.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/char_conv.c Sat Jul 19 03:10:10 2008 +0000 @@ -123,10 +123,7 @@ return g_strdup(QQ_NULL_MSG); } -/** - * @brief 把输入作为一个pascal字符串并返回一个用UFT-8转换的c-字符串.\n - * 返回已读入的字节数,或者当遇到错误时返回-1.该完成转换的UTF-8字符串被保存到ret中 - * +/* * take the input as a pascal string and return a converted c-string in UTF-8 * returns the number of bytes read, return -1 if fatal error * the converted UTF-8 will be saved in ret @@ -281,3 +278,17 @@ g_string_free(converted, FALSE); return ret; } + +void qq_filter_str(gchar *str) { + gchar *temp; + if (str == NULL) { + return; + } + + for (temp = str; *temp != 0; temp++) { + if (*temp == '\r' || *temp == '\n') *temp = 0x20; + } + g_strstrip(str); +} + +
--- a/libpurple/protocols/qq/char_conv.h Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/char_conv.h Sat Jul 19 03:10:10 2008 +0000 @@ -40,5 +40,5 @@ gchar *qq_encode_to_purple(guint8 *font_attr_data, gint len, const gchar *msg); gchar *qq_im_filter_html(const gchar *text); - +void qq_filter_str(gchar *str); #endif
--- a/libpurple/protocols/qq/group_conv.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/group_conv.c Sat Jul 19 03:10:10 2008 +0000 @@ -99,7 +99,9 @@ list = list->next; } - purple_conv_chat_add_users(PURPLE_CONV_CHAT(conv), names, NULL, flags, FALSE); + if (names != NULL && flags != NULL) { + purple_conv_chat_add_users(PURPLE_CONV_CHAT(conv), names, NULL, flags, FALSE); + } } /* clean up names */ while (names != NULL) {
--- a/libpurple/protocols/qq/group_free.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/group_free.c Sat Jul 19 03:10:10 2008 +0000 @@ -55,8 +55,10 @@ { g_return_if_fail(group != NULL); qq_group_free_member(group); + g_free(group->my_status_desc); g_free(group->group_name_utf8); g_free(group->group_desc_utf8); + g_free(group->notice_utf8); g_free(group); }
--- a/libpurple/protocols/qq/group_info.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/group_info.c Sat Jul 19 03:10:10 2008 +0000 @@ -87,7 +87,7 @@ /* only get online members when conversation window is on */ if (NULL == purple_find_conversation_with_account(PURPLE_CONV_TYPE_CHAT,group->group_name_utf8, purple_connection_get_account(gc))) { purple_debug(PURPLE_DEBUG_WARNING, "QQ", - "Conv windows for \"%s\" is not on, do not get online members\n", group->group_name_utf8); + "Conversation for \"%s\" is not open, ignore to get online members\n", group->group_name_utf8); return; } @@ -141,9 +141,6 @@ qq_send_group_cmd(gc, group, raw_data, bytes); } -/** - * @brief 处理群信息.当前群信息的处理还不完善,由于版本的不同导致协议的解读有差异. - */ void qq_process_group_cmd_get_group_info(guint8 *data, gint len, PurpleConnection *gc) { qq_group *group; @@ -157,6 +154,7 @@ guint32 unknown4; guint8 unknown1; gint bytes, num; + gchar *notice; g_return_if_fail(data != NULL && len > 0); qd = (qq_data *) gc->proto_data; @@ -186,22 +184,20 @@ bytes += qq_get32(&(group->group_category), data + bytes); bytes += qq_get16(&max_members, data + bytes); bytes += qq_get8(&unknown1, data + bytes); - /* XXX - * the following, while Eva: + /* the following, while Eva: * 4(unk), 4(verID), 1(nameLen), nameLen(qunNameContent), 1(0x00), * 2(qunNoticeLen), qunNoticeLen(qunNoticeContent, 1(qunDescLen), * qunDestLen(qunDestcontent)) */ bytes += qq_get8(&unknown1, data + bytes); - purple_debug(PURPLE_DEBUG_INFO, "QQ", "type=%u creatorid=%u category=%u\n", - group->group_type, group->creator_uid, group->group_category); - purple_debug(PURPLE_DEBUG_INFO, "QQ", "maxmembers=%u", max_members); + purple_debug(PURPLE_DEBUG_INFO, "QQ", "type=%u creatorid=%u category=%u maxmembers=%u\n", + group->group_type, group->creator_uid, group->group_category, max_members); /* strlen + <str content> */ bytes += convert_as_pascal_string(data + bytes, &(group->group_name_utf8), QQ_CHARSET_DEFAULT); purple_debug(PURPLE_DEBUG_INFO, "QQ", "group \"%s\"\n", group->group_name_utf8); bytes += qq_get16(&unknown, data + bytes); /* 0x0000 */ - bytes += convert_as_pascal_string(data + bytes, &(group->notice_utf8), QQ_CHARSET_DEFAULT); - purple_debug(PURPLE_DEBUG_INFO, "QQ", "notice \"%s\"\n", group->notice_utf8); + bytes += convert_as_pascal_string(data + bytes, ¬ice, QQ_CHARSET_DEFAULT); + purple_debug(PURPLE_DEBUG_INFO, "QQ", "notice \"%s\"\n", notice); bytes += convert_as_pascal_string(data + bytes, &(group->group_desc_utf8), QQ_CHARSET_DEFAULT); purple_debug(PURPLE_DEBUG_INFO, "QQ", "group_desc \"%s\"\n", group->group_desc_utf8); @@ -235,11 +231,16 @@ group->group_name_utf8, purple_connection_get_account(gc)); if(NULL == purple_conv) { purple_debug(PURPLE_DEBUG_WARNING, "QQ", - "Conv windows for \"%s\" is not on, do not set topic\n", group->group_name_utf8); + "Conversation for \"%s\" is not open, do not set topic\n", group->group_name_utf8); + return; } - else { - purple_conv_chat_set_topic(PURPLE_CONV_CHAT(purple_conv), NULL, group->notice_utf8); - } + + /* filter \r\n in notice */ + qq_filter_str(notice); + group->notice_utf8 = strdup(notice); + g_free(notice); + + purple_conv_chat_set_topic(PURPLE_CONV_CHAT(purple_conv), NULL, group->notice_utf8); } void qq_process_group_cmd_get_online_members(guint8 *data, gint len, PurpleConnection *gc) @@ -296,6 +297,7 @@ guint16 unknown; qq_group *group; qq_buddy *member; + gchar *nick; g_return_if_fail(data != NULL && len > 0); @@ -318,11 +320,24 @@ bytes += qq_get16(&(member->face), data + bytes); bytes += qq_get8(&(member->age), data + bytes); bytes += qq_get8(&(member->gender), data + bytes); - bytes += convert_as_pascal_string(data + bytes, &(member->nickname), QQ_CHARSET_DEFAULT); + bytes += convert_as_pascal_string(data + bytes, &nick, QQ_CHARSET_DEFAULT); bytes += qq_get16(&unknown, data + bytes); bytes += qq_get8(&(member->flag1), data + bytes); bytes += qq_get8(&(member->comm_flag), data + bytes); + /* filter \r\n in nick */ + qq_filter_str(nick); + member->nickname = g_strdup(nick); + g_free(nick); + + /* + if (QQ_DEBUG) { + purple_debug(PURPLE_DEBUG_INFO, "QQ", + "member [%09d]: flag1=0x%02x, comm_flag=0x%02x, nick=%s\n", + member_uid, member->flag1, member->comm_flag, member->nickname); + } + */ + member->last_refresh = time(NULL); } if(bytes > len) {
--- a/libpurple/protocols/qq/group_internal.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/group_internal.c Sat Jul 19 03:10:10 2008 +0000 @@ -230,9 +230,7 @@ *list = g_slist_remove(*list, GINT_TO_POINTER(id)); } -/** - * @brief Return the location of id in list, or NULL if not found (返回id在链表中的位置,没有找到则返回NULL) - */ +/* Return the location of id in list, or NULL if not found */ GSList *qq_get_pending_id(GSList *list, guint32 id) { return g_slist_find(list, GINT_TO_POINTER(id));
--- a/libpurple/protocols/qq/qq.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/qq.c Sat Jul 19 03:10:10 2008 +0000 @@ -749,7 +749,6 @@ NULL, NULL, NULL, - sizeof(PurplePluginProtocolInfo), /* struct_size */ NULL };
--- a/libpurple/protocols/qq/qq_network.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/qq_network.c Sat Jul 19 03:10:10 2008 +0000 @@ -189,13 +189,13 @@ return FALSE; /* set mask */ } -static gboolean packet_check_ack(qq_data *qd, guint16 seq) +static gboolean packet_check_ack(qq_data *qd, guint16 cmd, guint16 seq) { gpointer trans; g_return_val_if_fail(qd != NULL, FALSE); - trans = qq_send_trans_find(qd, seq); + trans = qq_send_trans_find(qd, cmd, seq); if (trans == NULL) { return FALSE; } @@ -360,7 +360,7 @@ /* ack packet, we need to update send tranactions */ /* we do not check duplication for server ack */ - is_reply = packet_check_ack(qd, seq); + is_reply = packet_check_ack(qd, cmd, seq); if ( !is_reply ) { if ( !qd->logged_in ) { /* packets before login */ @@ -460,7 +460,10 @@ return; } - gc->last_received = time(NULL); + /* keep alive will be sent in 30 seconds since last_receive + * QQ need a keep alive packet in every 60 seconds + gc->last_received = time(NULL); + */ purple_debug(PURPLE_DEBUG_INFO, "TCP_PENDING", "Read %d bytes from socket, rxlen is %d\n", buf_len, qd->tcp_rxlen); qd->tcp_rxqueue = g_realloc(qd->tcp_rxqueue, buf_len + qd->tcp_rxlen); @@ -565,7 +568,10 @@ return; } - gc->last_received = time(NULL); + /* keep alive will be sent in 30 seconds since last_receive + * QQ need a keep alive packet in every 60 seconds + gc->last_received = time(NULL); + */ if (buf_len < QQ_UDP_HEADER_LENGTH) { if (buf[0] != QQ_PACKET_TAG || buf[buf_len - 1] != QQ_PACKET_TAIL) { @@ -1240,7 +1246,6 @@ } if (QQ_DEBUG) { - qq_show_packet("QQ_SEND_CMD", buf, buf_len); purple_debug(PURPLE_DEBUG_INFO, "QQ", "<== [%05d], %s, total %d bytes is sent %d\n", seq, qq_get_cmd_desc(cmd), buf_len, bytes_sent);
--- a/libpurple/protocols/qq/qq_trans.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/qq_trans.c Sat Jul 19 03:10:10 2008 +0000 @@ -83,7 +83,7 @@ g_free(trans); } -gpointer qq_send_trans_find(qq_data *qd, guint16 seq) +gpointer qq_send_trans_find(qq_data *qd, guint16 cmd, guint16 seq) { GList *curr; GList *next; @@ -93,7 +93,7 @@ while(curr) { next = curr->next; trans = (transaction *) (curr->data); - if(trans->seq == seq) { + if(trans->cmd == cmd && trans->seq == seq) { return trans; } curr = next;
--- a/libpurple/protocols/qq/qq_trans.h Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/qq_trans.h Sat Jul 19 03:10:10 2008 +0000 @@ -30,7 +30,7 @@ void qq_send_trans_append(qq_data *qd, guint8 *buf, gint bus_len, guint16 cmd, guint16 seq); void qq_send_trans_remove(qq_data *qd, gpointer data); -gpointer qq_send_trans_find(qq_data *qd, guint16 seq); +gpointer qq_send_trans_find(qq_data *qd, guint16 cmd, guint16 seq); void qq_send_trans_remove_all(qq_data *qd); gint qq_send_trans_scan(qq_data *qd, gint *start, guint8 *buf, gint maxlen, guint16 *cmd, gint *retries);
--- a/libpurple/protocols/qq/send_file.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/qq/send_file.c Sat Jul 19 03:10:10 2008 +0000 @@ -156,7 +156,7 @@ gint size; /* FIXME: It seems that the transfer never use a packet * larger than 1500 bytes, so if it happened to be a - * larger packet, either error occurred or protocol should + * larger packet, either error occured or protocol should * be modified */ ft_info *info;
--- a/libpurple/protocols/yahoo/yahoo_profile.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/protocols/yahoo/yahoo_profile.c Sat Jul 19 03:10:10 2008 +0000 @@ -1282,6 +1282,10 @@ url_data = purple_util_fetch_url(url, TRUE, NULL, FALSE, yahoo_got_info, data); if (url_data != NULL) yd->url_datas = g_slist_prepend(yd->url_datas, url_data); + else { + g_free(data->name); + g_free(data); + } g_free(url); }
--- a/libpurple/win32/libc_interface.c Sat Jul 19 03:07:33 2008 +0000 +++ b/libpurple/win32/libc_interface.c Sat Jul 19 03:10:10 2008 +0000 @@ -327,9 +327,12 @@ case WSAETIMEDOUT: /* 10060 */ g_snprintf(errbuf, sizeof(errbuf), _("Connection timed out.")); break; - case WSAECONNREFUSED: /*10061 */ + case WSAECONNREFUSED: /* 10061 */ g_snprintf(errbuf, sizeof(errbuf), _("Connection refused.")); break; + case WSAEADDRINUSE: /* 10048 */ + g_snprintf(errbuf, sizeof(errbuf), _("Address already in use.")); + break; default: g_snprintf(errbuf, sizeof(errbuf), "Windows socket error #%d", errornum); }
--- a/pidgin/gtkaccount.c Sat Jul 19 03:07:33 2008 +0000 +++ b/pidgin/gtkaccount.c Sat Jul 19 03:10:10 2008 +0000 @@ -1186,7 +1186,6 @@ char *tmp; gboolean new_acct = FALSE, icon_change = FALSE; PurpleAccount *account; - PurplePluginProtocolInfo *prpl_info; /* Build the username string. */ username = g_strdup(gtk_entry_get_text(GTK_ENTRY(dialog->screenname_entry))); @@ -1254,8 +1253,7 @@ purple_account_set_alias(account, NULL); /* Buddy Icon */ - prpl_info = PURPLE_PLUGIN_PROTOCOL_INFO(dialog->plugin); - if (prpl_info != NULL && prpl_info->icon_spec.format != NULL) + if (dialog->prpl_info != NULL && dialog->prpl_info->icon_spec.format != NULL) { const char *filename;