From: Rhys Kidd <rhyskidd@gmail.com>
Date: Sun, 28 Jun 2015 04:06:30 +0000 (+0000)
Subject: Fix incorrect sizeof expression in syswrap-xen.c, reported by Coverity (#1233786)
X-Git-Tag: svn/VALGRIND_3_11_0~283
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9b3e7344ca418b30c4aed56ce942390e48c80082;p=thirdparty%2Fvalgrind.git

Fix incorrect sizeof expression in syswrap-xen.c, reported by Coverity (#1233786)
n-i-bz

Reviewed-by: Ian Campbell <ian.campbell@citrix.com>


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@15361
---

diff --git a/NEWS b/NEWS
index 3a6894816c..dd2069c971 100644
--- a/NEWS
+++ b/NEWS
@@ -252,6 +252,7 @@ n-i-bz  Fix compilation on distros with glibc < 2.5
 n-i-bz  (vex 3098) Avoid generation of Neon insns on non-Neon hosts
 n-i-bz  Enable rt_sigpending syscall on ppc64 linux.
 n-i-bz  mremap did not work properly on shared memory
+n-i-bz  Fix incorrect sizeof expression in syswrap-xen.c reported by Coverity
 
 
 Release 3.10.1 (25 November 2014)
diff --git a/coregrind/m_syswrap/syswrap-xen.c b/coregrind/m_syswrap/syswrap-xen.c
index a1e11c9294..43951039b7 100644
--- a/coregrind/m_syswrap/syswrap-xen.c
+++ b/coregrind/m_syswrap/syswrap-xen.c
@@ -943,7 +943,7 @@ PRE(tmem_op)
          *    vki_uint32_t subop;
          */
         PRE_MEM_READ("__HYPERVISOR_tmem_op pool_id",
-                     (Addr)&tmem->pool_id, sizeof(&tmem->pool_id));
+                     (Addr)&tmem->pool_id, sizeof(tmem->pool_id));
         PRE_XEN_TMEMOP_READ(ctrl, subop);
 
         switch (tmem->u.ctrl.subop) {