dvd-sector-copy.c: Check `n' nonnegative before comparing with unsigned.

Otherwise it might be coerced to unsigned, at which point the comparison
means something completely different.

diff --git a/dvd-sector-copy.c b/dvd-sector-copy.c
index c336d3d2f60500dd7bfb66c8dd574263b2463d42..177ce94f116577887bb351397dcdc5b51a028dcb 100644 (file)
--- a/dvd-sector-copy.c
+++ b/dvd-sector-copy.c
@@ -595,7 +595,7 @@ static ssize_t recovery_read(struct recoverybuf *r,
     printf(" -> %zd\n", n);
 #endif
     if (n != nn) {
     printf(" -> %zd\n", n);
 #endif
     if (n != nn) {

-      if (n > want) n = want;
+      if (n >= 0 && n > want) n = want;

       goto end;
     }
     r->start = pp;
       goto end;
     }
     r->start = pp;