Remove a cachePut that was adding FSObject to cache, which triggered out-of-memory errors. This code path was never previously triggered for filesystem-based storage.

Signed-off-by: Filip Navara <[email protected]>

Author: filipnavara

plumbing/format/packfile/packfile.go

@@ -79,7 +79,7 @@ func (p *Packfile) GetByOffset(o int64) (plumbing.EncodedObject, error) {
 		}
 	}
 
-	return p.objectAtOffset(o);
+	return p.objectAtOffset(o)
 }
 
 func (p *Packfile) objectHeaderAtOffset(offset int64) (*ObjectHeader, error) {
@@ -208,7 +208,7 @@ func (p *Packfile) objectAtOffset(offset int64) (plumbing.EncodedObject, error)
 
 	// If we have no filesystem, we will return a MemoryObject instead
 	// of an FSObject.
-	if p.fs == nil || h.Length <= 16 * 1024 {
+	if p.fs == nil || h.Length <= 16*1024 {
 		return p.getNextObject(h)
 	}
 
@@ -347,8 +347,6 @@ func (p *Packfile) fillOFSDeltaObjectContent(obj plumbing.EncodedObject, offset
 		if err != nil {
 			return err
 		}
-
-		p.cachePut(base)
 	}
 
 	obj.SetType(base.Type())