Mercurial > pidgin
diff libpurple/protocols/msn/slpmsg_part.c @ 31045:f91501dc406e
Remove extra variables, and this sizeof() is wrong.
author | Elliott Sales de Andrade <qulogic@pidgin.im> |
---|---|
date | Fri, 17 Dec 2010 07:58:33 +0000 |
parents | 7b771e6f1142 |
children | a8cc50c2279f |
line wrap: on
line diff
--- 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; }