dvd-sector-copy.c: Fix final recovery.

Write this the other way around, because it's easier to read that way.
And then notice that it's broken because it doesn't actually set the
return value properly.

diff --git a/dvd-sector-copy.c b/dvd-sector-copy.c
index dd97d8b88bfe7de17bc53491b2d507a125989f91..98ee8ec83881c590cd4f9f8d6e760ce09e74a1df 100644 (file)
--- a/dvd-sector-copy.c
+++ b/dvd-sector-copy.c
@@ -710,10 +710,12 @@ static ssize_t find_good_sector(secaddr *pos_inout, secaddr end,
     else bad_hi = pos + n + 1;
   }
   recovered(bad_lo, bad_hi); *pos_inout = good;
     else bad_hi = pos + n + 1;
   }
   recovered(bad_lo, bad_hi); *pos_inout = good;

-  if (r.pos + r.start <= good && good <= r.pos + r.end) {
-    rearrange_sectors(&r, 0, good - r.pos, r.pos + r.end - good);
-  } else
+  if (good < r.pos + r.start || r.pos + r.end <= good)

     n = 0;
     n = 0;

+  else {
+    n = r.pos + r.end - good;
+    rearrange_sectors(&r, 0, good - r.pos, n);
+  }

   return (n);
 }
 
   return (n);
 }