[libvirt] [PATCH] xen: fix build error on rhel-5

Eric Blake eblake at redhat.com
Tue Apr 14 13:03:53 UTC 2015


On 04/14/2015 06:43 AM, Pavel Hrdina wrote:
> ../../src/xen/block_stats.c:82: warning: dereferencing type-punned
>     pointer will break strict-aliasing rules [-Wstrict-aliasing]
> 
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>  src/xen/block_stats.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/xen/block_stats.c b/src/xen/block_stats.c
> index e1e10a2..9dd3a07 100644
> --- a/src/xen/block_stats.c
> +++ b/src/xen/block_stats.c
> @@ -63,7 +63,7 @@ static int64_t
>  read_stat(const char *path)
>  {
>      char str[64];
> -    int64_t r;
> +    long long r;
>      size_t i;
>      FILE *fp;
>  
> @@ -79,10 +79,10 @@ read_stat(const char *path)
>          return -1;
>  
>      str[i] = '\0';              /* make sure the string is nul-terminated */
> -    if (virStrToLong_ll(str, NULL, 10, (long long *) &r) < 0)
> +    if (virStrToLong_ll(str, NULL, 10, &r) < 0)
>          return -1;
>  
> -    return r;
> +    return (int64_t)r;

The cast is not necessary here.

ACK whether or not you drop the cast.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150414/73c95db1/attachment-0001.sig>


More information about the libvir-list mailing list