memchecker.h: fix memchecker no-data case

jsquyres · jsquyres · commit 7b73c868d567 · 2016-02-18T10:48:11.000-08:00
Thanks to @clintonstimpson for reporting the issue. Fixes #100 Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
diff --git a/ompi/include/ompi/memchecker.h b/ompi/include/ompi/memchecker.h
@@ -10,6 +10,7 @@
  *                         and Technology (RIST). All rights reserved.
  * Copyright (c) 2014      Intel, Inc. All rights reserved.
  *
+ * Copyright (c) 2016 Cisco Systems, Inc.  All rights reserved.
  * $COPYRIGHT$
  *
  * Additional copyrights may follow
@@ -100,6 +101,10 @@ static inline int memchecker_call (int (*f)(void *, size_t), const void * addr,
         return OMPI_SUCCESS;
     }
 
+    if ((0 == count) || (0 == datatype->super.size)) {
+        return OMPI_SUCCESS;
+    }
+
     if( datatype->super.size == (size_t) (datatype->super.true_ub - datatype->super.true_lb) ) {
         /*  We have a contiguous type. */
         f( (void*)addr , datatype->super.size * count );

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,7 @@`
`10`	`10`	`* and Technology (RIST). All rights reserved.`
`11`	`11`	`* Copyright (c) 2014 Intel, Inc. All rights reserved.`
`12`	`12`	`*`
	`13`	`+ * Copyright (c) 2016 Cisco Systems, Inc. All rights reserved.`
`13`	`14`	`* $COPYRIGHT$`
`14`	`15`	`*`
`15`	`16`	`* Additional copyrights may follow`
`@@ -100,6 +101,10 @@ static inline int memchecker_call (int (f)(void , size_t), const void * addr,`
`100`	`101`	`return OMPI_SUCCESS;`
`101`	`102`	`}`
`102`	`103`
	`104`	`+ if ((0 == count) \|\| (0 == datatype->super.size)) {`
	`105`	`+ return OMPI_SUCCESS;`
	`106`	`+ }`
	`107`	`+`
`103`	`108`	`if( datatype->super.size == (size_t) (datatype->super.true_ub - datatype->super.true_lb) ) {`
`104`	`109`	`/* We have a contiguous type. */`
`105`	`110`	`f( (void)addr , datatype->super.size count );`