fsck: detect gitmodules files
[gitweb.git] / vcs-svn / sliding_window.c
index 1b8d9875ed8831a98d0107db71e0ba1c3083a714..06d273c9e8ce2b6b8ddbf967693e5305355d5be6 100644 (file)
@@ -12,7 +12,7 @@ static int input_error(struct line_buffer *file)
 {
        if (!buffer_ferror(file))
                return error("delta preimage ends early");
-       return error("cannot read delta preimage: %s", strerror(errno));
+       return error_errno("cannot read delta preimage");
 }
 
 static int skip_or_whine(struct line_buffer *file, off_t gap)
@@ -31,15 +31,15 @@ static int read_to_fill_or_whine(struct line_buffer *file,
        return 0;
 }
 
-static int check_overflow(off_t a, size_t b)
+static int check_offset_overflow(off_t offset, uintmax_t len)
 {
-       if (b > maximum_signed_value_of_type(off_t))
+       if (len > maximum_signed_value_of_type(off_t))
                return error("unrepresentable length in delta: "
-                               "%"PRIuMAX" > OFF_MAX", (uintmax_t) b);
-       if (signed_add_overflows(a, (off_t) b))
+                               "%"PRIuMAX" > OFF_MAX", len);
+       if (signed_add_overflows(offset, (off_t) len))
                return error("unrepresentable offset in delta: "
                                "%"PRIuMAX" + %"PRIuMAX" > OFF_MAX",
-                               (uintmax_t) a, (uintmax_t) b);
+                               (uintmax_t) offset, len);
        return 0;
 }
 
@@ -48,12 +48,14 @@ int move_window(struct sliding_view *view, off_t off, size_t width)
        off_t file_offset;
        assert(view);
        assert(view->width <= view->buf.len);
-       assert(!check_overflow(view->off, view->buf.len));
+       assert(!check_offset_overflow(view->off, view->buf.len));
 
-       if (check_overflow(off, width))
+       if (check_offset_overflow(off, width))
                return -1;
        if (off < view->off || off + width < view->off + view->width)
                return error("invalid delta: window slides left");
+       if (view->max_off >= 0 && view->max_off < off + (off_t) width)
+               return error("delta preimage ends early");
 
        file_offset = view->off + view->buf.len;
        if (off < file_offset) {