Re: [Qemu-devel] [PATCH 1/2] qemu-io: small cleanups for the aio code

[Anthony Liguori]

Christoph Hellwig wrote:
>         if (ctx->Pflag) {
> -               void *cmp_buf = malloc(total);
> +               void *cmp_buf = malloc(ctx->qiov.size);
>  
> -		memset(cmp_buf, ctx->pattern, total);
> -               if (memcmp(ctx->buf, cmp_buf, total)) {
> +               memset(cmp_buf, ctx->pattern, ctx->qiov.size);
> +               if (memcmp(ctx->buf, cmp_buf, ctx->qiov.size)) {
>                         printf("Pattern verification failed at offset %lld, "
>                                 "%d bytes\n",
> -                               (long long) ctx->offset, total);
> +                               (long long) ctx->offset, ctx->qiov.size);
>   

This breaks the build because ctx->qiov.size is a size_t and your format 
parameter is a %d.

Regards,

Anthony Liguori