changeset
Valgrind pointed out that dest and src may overlap, so we need to use
Wed, 05 May 2010 09:07:10 +0000
- author
- Elliott Sales de Andrade <qulogic@pidgin.im>
- date
- Wed, 05 May 2010 09:07:10 +0000
- changeset 30107
- 8f91902bcc39
- parent 30106
- 142f2a062877
- child 30108
- 00eddf66bdd8
Valgrind pointed out that dest and src may overlap, so we need to use
g_memmove here instead of memcpy.
| libpurple/protocols/msn/directconn.c | file | annotate | diff | comparison | revisions |
--- a/libpurple/protocols/msn/directconn.c Wed May 05 08:18:35 2010 +0000 +++ b/libpurple/protocols/msn/directconn.c Wed May 05 09:07:10 2010 +0000 @@ -710,7 +710,7 @@ } if (dc->in_pos > packet_length + 4) { - memcpy(dc->in_buffer, dc->in_buffer + 4 + packet_length, dc->in_pos - packet_length - 4); + g_memmove(dc->in_buffer, dc->in_buffer + 4 + packet_length, dc->in_pos - packet_length - 4); } dc->in_pos -= packet_length + 4;
