Changeset 39394 for trunk/Ohana/src/libfits/table/F_set_column.c

Timestamp:

Feb 26, 2016, 6:19:43 AM (10 years ago)

Author:

eugene

Message:

trying to catch some (possible) stack and heap overflow errors: adding padding to allocated config regions

File:

• trunk/Ohana/src/libfits/table/F_set_column.c (modified) (6 diffs)

trunk/Ohana/src/libfits/table/F_set_column.c

@@ -2 +2 @@
 # include <gfitsio.h>
 # define OHANA_MEMCHECK 0
+# define CHECK_MEMBLOCKS 0
 
 # define SWAP_NONE 
@@ -45 +46 @@
 # endif
 
+  memset (tlabel,  0, 256);
+  memset (field,   0, 256);
+  memset (format,  0, 256);
+  memset (type,    0, 64);
+  memset (tmpline, 0, 64);
+
   if (label == (char *) NULL) return (FALSE);
   if (label[0] == 0) return (FALSE);
@@ -95 +102 @@
   }
 
+  // ohana_memcheck (TRUE);
+
   /* make duplicate of data with correct type
      byte swap and Bzero/Bscale */
   ALLOCATE (array, char, Nbytes*Nval*Nrow);
+  if (CHECK_MEMBLOCKS) {
+    OhanaMemblock *ref = (OhanaMemblock *) array - 1;
+    fprintf (stderr, "ref: "OFF_T_FMT", array: "OFF_T_FMT"\n", (off_t) ref, (off_t) array);
+    if (!ref->nextBlock && !ref->prevBlock) abort();
+  }
+
   Pin = data;
   Pout = array;
+
+  // ohana_memcheck (TRUE);
+
+  if (CHECK_MEMBLOCKS) {
+    OhanaMemblock *ref = (OhanaMemblock *) array - 1;
+    fprintf (stderr, "ref: "OFF_T_FMT", array: "OFF_T_FMT"\n", (off_t) ref, (off_t) array);
+    if (!ref->nextBlock && !ref->prevBlock) abort();
+  }
 
   // does it makes sense to scale 'char' data?
@@ -187 +210 @@
   }
 
+  if (CHECK_MEMBLOCKS) {
+    OhanaMemblock *ref = (OhanaMemblock *) array - 1;
+    fprintf (stderr, "ref: "OFF_T_FMT", array: "OFF_T_FMT"\n", (off_t) ref, (off_t) array);
+    if (!ref->nextBlock && !ref->prevBlock) abort();
+  }
+
   /* check array space */
   if (Nx*Ny < Nx*(Nrow - 1) + Nstart + Nval*Nbytes) {
@@ -193 +222 @@
   }
 
+  // ohana_memcheck (TRUE);
   /* insert bytes from array into appropriate section of buffer */
   Pout = table[0].buffer + Nstart;
@@ -206 +236 @@
   myAssert (   type[63]  == 0x7f, "oops");
   myAssert (tmpline[63]  == 0x7f, "oops");
-
- ohana_memcheck (TRUE);
-# endif
+# endif
+  // ohana_memcheck (TRUE);
+
+  if (CHECK_MEMBLOCKS) {
+    OhanaMemblock *ref = (OhanaMemblock *) array - 1;
+    fprintf (stderr, "ref: "OFF_T_FMT", array: "OFF_T_FMT"\n", (off_t) ref, (off_t) array);
+    if (!ref->nextBlock && !ref->prevBlock) abort();
+  }
 
   free (array);