# HG changeset patch # User Elliott Sales de Andrade # Date 1292572713 0 # Node ID f91501dc406e4011d54c22129b04da31682bc1a7 # Parent 7b771e6f1142f548dbadc4910b84253524d5682b Remove extra variables, and this sizeof() is wrong. diff -r 7b771e6f1142 -r f91501dc406e libpurple/protocols/msn/slpmsg_part.c --- a/libpurple/protocols/msn/slpmsg_part.c Fri Dec 17 07:49:47 2010 +0000 +++ b/libpurple/protocols/msn/slpmsg_part.c Fri Dec 17 07:58:33 2010 +0000 @@ -48,20 +48,17 @@ MsnSlpMessagePart *msn_slpmsgpart_new_from_data(const char *data, size_t data_len) { MsnSlpMessagePart *part; - MsnP2PHeader *header; - const char *tmp; int body_len; - if (data_len < sizeof(*header)) { + if (data_len < P2P_PACKET_HEADER_SIZE) { return NULL; } part = msn_slpmsgpart_new(NULL, NULL); - tmp = data; /* Extract the binary SLP header */ - part->header = msn_p2p_header_from_wire(tmp); - tmp += P2P_PACKET_HEADER_SIZE; + part->header = msn_p2p_header_from_wire(data); + data += P2P_PACKET_HEADER_SIZE; /* Extract the body */ body_len = data_len - P2P_PACKET_HEADER_SIZE - P2P_PACKET_FOOTER_SIZE; @@ -70,13 +67,13 @@ if (body_len > 0) { part->size = body_len; part->buffer = g_malloc(body_len); - memcpy(part->buffer, tmp, body_len); - tmp += body_len; + memcpy(part->buffer, data, body_len); + data += body_len; } /* Extract the footer */ if (body_len >= 0) - part->footer = msn_p2p_footer_from_wire(tmp); + part->footer = msn_p2p_footer_from_wire(data); return part; }