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

[Anthony Liguori <anthony@codemonkey.ws>]

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