Changeset 577c4080 in rtems

Timestamp:

May 17, 1999, 3:28:52 PM (22 years ago)

Author:

Joel Sherrill <joel.sherrill@…>

Branches:

4.10, 4.11, 4.8, 4.9, 5, master

Children:

a238cc9

Parents:

9e7d42dd

Message:

Patch from D. V. Henkel-Wallace <gumby@…>:

Here's a tiny patch that shreds memory returned to the pool (such as by
free() and delete). This may help people find some nasty
bugs, so here it is.

Files:

• c/src/exec/rtems/src/region.c (modified) (3 diffs)
• cpukit/rtems/src/region.c (modified) (3 diffs)

c/src/exec/rtems/src/region.c

@@ -13 +13 @@
  *  $Id$
  */
+
+#ifdef RTEMS_REGION_FREE_SHRED_PATTERN
+#include <string.h>
+#endif
 
 #include <rtems/system.h>
@@ -451 +455 @@
   Objects_Locations        location;
   void                   **the_segment;
+#ifdef RTEMS_SHRED_ON_FREE
+  unsigned32               size;
+#endif
   int                      status;
 
@@ -465 +472 @@
 
       _Region_Debug_Walk( the_region, 3 );
+
+#ifdef RTEMS_REGION_FREE_SHRED_PATTERN
+      if ( _Heap_Size_of_user_area( &the_region->Memory, segment, size ) ) {
+        memset(segment, (RTEMS_REGION_FREE_SHRED_BYTE & 0xFF), size);
+      } else {
+        _Thread_Enable_dispatch();
+        return RTEMS_INVALID_ADDRESS;
+      }
+#endif
 
       status = _Region_Free_segment( the_region, segment );

cpukit/rtems/src/region.c

@@ -13 +13 @@
  *  $Id$
  */
+
+#ifdef RTEMS_REGION_FREE_SHRED_PATTERN
+#include <string.h>
+#endif
 
 #include <rtems/system.h>
@@ -451 +455 @@
   Objects_Locations        location;
   void                   **the_segment;
+#ifdef RTEMS_SHRED_ON_FREE
+  unsigned32               size;
+#endif
   int                      status;
 
@@ -465 +472 @@
 
       _Region_Debug_Walk( the_region, 3 );
+
+#ifdef RTEMS_REGION_FREE_SHRED_PATTERN
+      if ( _Heap_Size_of_user_area( &the_region->Memory, segment, size ) ) {
+        memset(segment, (RTEMS_REGION_FREE_SHRED_BYTE & 0xFF), size);
+      } else {
+        _Thread_Enable_dispatch();
+        return RTEMS_INVALID_ADDRESS;
+      }
+#endif
 
       status = _Region_Free_segment( the_region, segment );