vma-iter: Avoid empty intervals.

* lib/vma-iter.c (vma_iterate) [IRIX, OSF/1]: Don't call the callback
on an empty interval.

diff --git a/ChangeLog b/ChangeLog
index bea7d1e5c67633668df3f26af4c014c7f6ea571c..ab1ad1e7b5b083b6e74f80bbd5d35e5aec905621 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2011-01-24  Bruno Haible  <bruno@clisp.org>
+
+       vma-iter: Avoid empty intervals.
+       * lib/vma-iter.c (vma_iterate) [IRIX, OSF/1]: Don't call the callback
+       on an empty interval.
+
 2011-01-24  Jim Meyering  <meyering@redhat.com>
 
        u64: remove unnecessary #include

diff --git a/lib/vma-iter.c b/lib/vma-iter.c
index 251ef755404397aea4f872d99834311257105d0b..8ef865de7ffd237cd5b12baed5493fd841cdfa31 100644 (file)
--- a/lib/vma-iter.c
+++ b/lib/vma-iter.c
@@ -331,10 +331,12 @@ vma_iterate (vma_iterate_callback_fn callback, void *data)
         {
           /* Consider [start,end-1] \ [auxmap_start,auxmap_end-1]
              = [start,auxmap_start-1] u [auxmap_end,end-1].  */
-          if (callback (data, start, auxmap_start, flags))
-            break;
-          if (callback (data, auxmap_end, end, flags))
-            break;
+          if (start < auxmap_start)
+            if (callback (data, start, auxmap_start, flags))
+              break;
+          if (auxmap_end - 1 < end - 1)
+            if (callback (data, auxmap_end, end, flags))
+              break;
         }
       else
         {