[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen stable-4.2] tools: Fix memset(&p, 0, sizeof(p)) idiom in several places.



commit 3f5e3cd97398468d624cf907979c2bb12ff7ee7e
Author:     Michael Young <m.a.young@xxxxxxxxxxxx>
AuthorDate: Wed Feb 13 17:00:15 2013 +0000
Commit:     Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
CommitDate: Thu Feb 28 11:45:47 2013 +0000

    tools: Fix memset(&p,0,sizeof(p)) idiom in several places.
    
    gcc 4.8 identifies several places where code of the form memset(x, 0,
    sizeof(x)); is used incorrectly, meaning that less memory is set to
    zero than required.
    
    Signed-off-by: Michael Young <m.a.young@xxxxxxxxxxxx>
    Committed-by: Keir Fraser <keir@xxxxxxx>
    (cherry picked from commit d119301b5816b39b5ba722a2f8b301b37e8e34bd)
---
 tools/blktap2/drivers/md5.c                    |    2 +-
 tools/debugger/kdd/kdd-xen.c                   |    2 +-
 tools/libxc/xc_dom_boot.c                      |    2 +-
 tools/libxl/libxl_qmp.c                        |    2 +-
 tools/python/xen/lowlevel/netlink/libnetlink.c |    2 +-
 tools/xenstat/libxenstat/src/xenstat_linux.c   |    2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/blktap2/drivers/md5.c b/tools/blktap2/drivers/md5.c
index 27a360e..e765832 100644
--- a/tools/blktap2/drivers/md5.c
+++ b/tools/blktap2/drivers/md5.c
@@ -174,7 +174,7 @@ void MD5Final(unsigned char digest[16], struct MD5Context 
*ctx)
     MD5Transform(ctx->buf, (uint32_t *) ctx->in);
     byteReverse((unsigned char *) ctx->buf, 4);
     memcpy(digest, ctx->buf, 16);
-    memset(ctx, 0, sizeof(ctx));     /* In case it's sensitive */
+    memset(ctx, 0, sizeof(*ctx));     /* In case it's sensitive */
 }
 
 /* The four core functions - F1 is optimized somewhat */
diff --git a/tools/debugger/kdd/kdd-xen.c b/tools/debugger/kdd/kdd-xen.c
index b2488c8..4fbea7d 100644
--- a/tools/debugger/kdd/kdd-xen.c
+++ b/tools/debugger/kdd/kdd-xen.c
@@ -333,7 +333,7 @@ int kdd_get_regs(kdd_guest *g, int cpuid, kdd_regs *r, int 
w64)
     if (!cpu) 
         return -1;
 
-    memset(r, 0, sizeof(r));
+    memset(r, 0, sizeof(*r));
     
     if (w64)
         kdd_get_regs_x86_64(cpu, &r->r64);
diff --git a/tools/libxc/xc_dom_boot.c b/tools/libxc/xc_dom_boot.c
index a9a868c..438852b 100644
--- a/tools/libxc/xc_dom_boot.c
+++ b/tools/libxc/xc_dom_boot.c
@@ -266,7 +266,7 @@ int xc_dom_boot_image(struct xc_dom_image *dom)
         return rc;
 
     /* let the vm run */
-    memset(ctxt, 0, sizeof(ctxt));
+    memset(ctxt, 0, sizeof(*ctxt));
     if ( (rc = dom->arch_hooks->vcpu(dom, ctxt)) != 0 )
         return rc;
     xc_dom_unmap_all(dom);
diff --git a/tools/libxl/libxl_qmp.c b/tools/libxl/libxl_qmp.c
index b4cc247..dee9503 100644
--- a/tools/libxl/libxl_qmp.c
+++ b/tools/libxl/libxl_qmp.c
@@ -378,7 +378,7 @@ static int qmp_open(libxl__qmp_handler *qmp, const char 
*qmp_socket_path,
     ret = libxl_fd_set_cloexec(qmp->ctx, qmp->qmp_fd, 1);
     if (ret) return -1;
 
-    memset(&qmp->addr, 0, sizeof (&qmp->addr));
+    memset(&qmp->addr, 0, sizeof (qmp->addr));
     qmp->addr.sun_family = AF_UNIX;
     strncpy(qmp->addr.sun_path, qmp_socket_path,
             sizeof (qmp->addr.sun_path));
diff --git a/tools/python/xen/lowlevel/netlink/libnetlink.c 
b/tools/python/xen/lowlevel/netlink/libnetlink.c
index 75e4a51..c5fc0bf 100644
--- a/tools/python/xen/lowlevel/netlink/libnetlink.c
+++ b/tools/python/xen/lowlevel/netlink/libnetlink.c
@@ -37,7 +37,7 @@ int rtnl_open_byproto(struct rtnl_handle *rth, unsigned 
subscriptions,
        int sndbuf = 32768;
        int rcvbuf = 32768;
 
-       memset(rth, 0, sizeof(rth));
+       memset(rth, 0, sizeof(*rth));
 
        rth->fd = socket(AF_NETLINK, SOCK_RAW, protocol);
        if (rth->fd < 0) {
diff --git a/tools/xenstat/libxenstat/src/xenstat_linux.c 
b/tools/xenstat/libxenstat/src/xenstat_linux.c
index 1543fb2..931b24e 100644
--- a/tools/xenstat/libxenstat/src/xenstat_linux.c
+++ b/tools/xenstat/libxenstat/src/xenstat_linux.c
@@ -113,7 +113,7 @@ int parseNetDevLine(char *line, char *iface, unsigned long 
long *rxBytes, unsign
 
        /* Initialize all variables called has passed as non-NULL to zeros */
        if (iface != NULL)
-               memset(iface, 0, sizeof(iface));
+               memset(iface, 0, sizeof(*iface));
        if (rxBytes != NULL)
                *rxBytes = 0;
        if (rxPackets != NULL)
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.2

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.