anvil: Penalty tracking was moving last checksums wrongly with memcpy().

Depending on the OS/etc this could have caused the checksum tracking to go
wrong. Found by Coverity.

diff --git a/src/anvil/penalty.c b/src/anvil/penalty.c
index dd5bfa1187500050df62c5a97c629e1dd650540d..5034421a8cec31b1f40c626fa83a7b6f357e9ee5 100644 (file)
--- a/src/anvil/penalty.c
+++ b/src/anvil/penalty.c
@@ -105,8 +105,8 @@ penalty_bump_checksum(struct penalty_rec *rec, unsigned int checksum)
        for (i = 0; i < count; i++) {
                if (checksums[i] == checksum) {
                        if (i > 0) {
-                               memcpy(checksums + 1, checksums,
-                                      sizeof(checksums[0]) * i);
+                               memmove(checksums + 1, checksums,
+                                       sizeof(checksums[0]) * i);
                                checksums[0] = checksum;
                        }
                        return TRUE;