aboutsummaryrefslogtreecommitdiff
path: root/objectstore/loose/parse.go
diff options
context:
space:
mode:
authorGravatar Runxi Yu2026-02-21 03:57:20 +0800
committerGravatar Runxi Yu2026-02-21 03:57:20 +0800
commiteeadac1851149248e788579cc83b925c27c2a8e1 (patch)
tree16e96a0d9d344e4f728d7759006b846cf95af688 /objectstore/loose/parse.go
parentobjectstore: Clarify the hash contract in ReadBytesFull (diff)
signatureNo signature
objectstore/loose: Verify the length where possible.
Diffstat (limited to 'objectstore/loose/parse.go')
-rw-r--r--objectstore/loose/parse.go8
1 files changed, 6 insertions, 2 deletions
diff --git a/objectstore/loose/parse.go b/objectstore/loose/parse.go
index 30c7e251..fc678262 100644
--- a/objectstore/loose/parse.go
+++ b/objectstore/loose/parse.go
@@ -23,11 +23,15 @@ func decodeAll(file *os.File) ([]byte, error) {
// parseRaw parses a loose object payload in "type size\0content" format.
func parseRaw(raw []byte) (objecttype.Type, []byte, error) {
- ty, _, headerLen, ok := objectheader.Parse(raw)
+ ty, size, headerLen, ok := objectheader.Parse(raw)
if !ok {
return objecttype.TypeInvalid, nil, errors.New("objectstore/loose: malformed object header")
}
- return ty, raw[headerLen:], nil
+ content := raw[headerLen:]
+ if int64(len(content)) != size {
+ return objecttype.TypeInvalid, nil, errors.New("objectstore/loose: object header size/content mismatch")
+ }
+ return ty, content, nil
}
// readHeader reads and parses a loose object header from br.