cedric pushed a commit to branch master.

commit 4a584fcc325f7a96513865ee194bd4c7d3c1a7d7
Author: Cedric BAIL <[email protected]>
Date:   Fri Mar 22 20:11:43 2013 +0900

    eina: correctly advertise to valgrind Eina_Cow memory size.
---
 src/lib/eina/eina_cow.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/lib/eina/eina_cow.c b/src/lib/eina/eina_cow.c
index 5512590..0b5514c 100644
--- a/src/lib/eina/eina_cow.c
+++ b/src/lib/eina/eina_cow.c
@@ -357,18 +357,18 @@ eina_cow_free(Eina_Cow *cow, const Eina_Cow_Data *data)
 
    ref = EINA_COW_PTR_GET(data);
 #ifndef NVALGRIND
-   VALGRIND_MAKE_MEM_DEFINED(ref, sizeof (ref));
+   VALGRIND_MAKE_MEM_DEFINED(ref, sizeof (*ref));
 #endif
    ref->refcount--;
 
-   if (ref->refcount > 0) return ;
+   if (ref->refcount > 0)
+     {
 #ifndef NVALGRIND
-   VALGRIND_MAKE_MEM_NOACCESS(ref, sizeof (ref));
+       VALGRIND_MAKE_MEM_NOACCESS(ref, sizeof (*ref));
 #endif
+       return ;
+     }
 
-#ifndef NVALGRIND
-   VALGRIND_MAKE_MEM_DEFINED(ref, sizeof (ref));
-#endif
 #ifdef EINA_COW_MAGIC_ON
    EINA_MAGIC_SET(ref, EINA_MAGIC_NONE);
 #endif

-- 

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar

Reply via email to