From 98231f518208f77ee6e92291976a1512ad1da282 Mon Sep 17 00:00:00 2001 From: Runxi Yu Date: Sun, 22 Mar 2026 23:23:32 +0000 Subject: refstore/files: UB on use after close --- refstore/files/transaction_commit.go | 15 --------------- 1 file changed, 15 deletions(-) (limited to 'refstore/files/transaction_commit.go') diff --git a/refstore/files/transaction_commit.go b/refstore/files/transaction_commit.go index eba959cb..dae4d8ee 100644 --- a/refstore/files/transaction_commit.go +++ b/refstore/files/transaction_commit.go @@ -6,15 +6,8 @@ import ( ) func (tx *Transaction) Commit() error { - err := tx.ensureOpen() - if err != nil { - return err - } - prepared, err := tx.prepare() if err != nil { - tx.closed = true - return err } @@ -29,16 +22,12 @@ func (tx *Transaction) Commit() error { err = tx.writeLoose(item) if err != nil { - tx.closed = true - return err } } err = tx.applyPackedDeletes(prepared) if err != nil { - tx.closed = true - return err } @@ -48,8 +37,6 @@ func (tx *Transaction) Commit() error { if item.target.ref.isLoose { err = tx.store.rootFor(item.target.loc.root).Remove(item.target.loc.path) if err != nil && !errors.Is(err, os.ErrNotExist) { - tx.closed = true - return err } @@ -59,7 +46,5 @@ func (tx *Transaction) Commit() error { } } - tx.closed = true - return nil } -- cgit v1.3.1-10-gc9f91