From 2002-01-18 Greg McGary <greg@mcgary.org>:

* (create_mem_region): Disallow useless empty region.  Regions are
half-open intervals, so allow [A..B) [B..C) as non-overlapping.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 6d826dbd2f3fb6b3112656f52c4243db4a106aa6..6eaad95b2ae83986395c035f6fd4716eec958698 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2002-02-13  Andrew Cagney  <ac131313@redhat.com>
+
+       From 2002-01-18 Greg McGary <greg@mcgary.org>:
+       * (create_mem_region): Disallow useless empty region.  Regions are
+       half-open intervals, so allow [A..B) [B..C) as non-overlapping.
+
 2002-02-13  Michael Chastain <mec@shout.net>
 
        * defs.h: Kill CONST_PTR.

diff --git a/gdb/memattr.c b/gdb/memattr.c
index e8eb56a458b17f91b70337b94b62b1cd19a3fd02..8d7b0d8c558eb5a7ae91b9fc3ed4ed3e7608877f 100644 (file)
--- a/gdb/memattr.c
+++ b/gdb/memattr.c
@@ -45,9 +45,10 @@ create_mem_region (CORE_ADDR lo, CORE_ADDR hi,
 {
   struct mem_region *n, *new;
 
-  if (lo > hi)
+  /* lo == hi is a useless empty region */
+  if (lo >= hi)
     {
-      printf_unfiltered ("invalid memory region\n");
+      printf_unfiltered ("invalid memory region: low >= high\n");
       return NULL;
     }
 
@@ -55,8 +56,8 @@ create_mem_region (CORE_ADDR lo, CORE_ADDR hi,
   while (n)
     {
       /* overlapping node */
-      if ((lo >= n->lo && lo <= n->hi) ||
-         (hi >= n->lo && hi <= n->hi))
+      if ((lo >= n->lo && lo < n->hi) ||
+         (hi > n->lo && hi <= n->hi))
        {
          printf_unfiltered ("overlapping memory region\n");
          return NULL;