memmove already deal with any alignment so there's no reason to have row byte pointer cast to uint32_t. Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> --- common/canvas_base.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/common/canvas_base.c b/common/canvas_base.c index 5815e9c..5b3649d 100644 --- a/common/canvas_base.c +++ b/common/canvas_base.c @@ -522,9 +522,9 @@ static void canvas_fix_alignment(uint8_t *bits, int row; uint8_t *dest = bits; for (row = height - 1; row > 0; --row) { - uint32_t *dest_aligned, *dest_misaligned; - dest_aligned = SPICE_ALIGNED_CAST(uint32_t *,dest + stride_pixman*row); - dest_misaligned = SPICE_UNALIGNED_CAST(uint32_t*,dest + stride_encoded*row); + uint8_t *dest_aligned, *dest_misaligned; + dest_aligned = dest + stride_pixman*row; + dest_misaligned = dest + stride_encoded*row; memmove(dest_aligned, dest_misaligned, stride_encoded); } } -- 2.9.4 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel