| To: | Mark Tinguely <tinguely@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2/2] xfs free the list of recovery items on error. |
| From: | Ben Myers <bpm@xxxxxxx> |
| Date: | Thu, 5 Dec 2013 16:52:04 -0600 |
| Cc: | xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20131204205105.GV1935@xxxxxxx> |
| References: | <20131002125110.745269864@xxxxxxx> <20131002125409.930462395@xxxxxxx> <20131204205105.GV1935@xxxxxxx> |
| User-agent: | Mutt/1.5.20 (2009-06-14) |
On Wed, Dec 04, 2013 at 02:51:05PM -0600, Ben Myers wrote: > On Wed, Oct 02, 2013 at 07:51:12AM -0500, Mark Tinguely wrote: > > Recovery builds a list of items on the transaction's > > r_itemq head. Normally these items are committed and freed. > > But in the event of a recovery error, these allocations > > are leaked. > > > > If the error occurs during item reordering, then reconstruct > > the r_itemq list before deleting the list to avoid leaking > > the entries that were on one of the temporary lists. > > > > Signed-off-by: Mark Tinguely <tinguely@xxxxxxx> > > Looks good to me. > Reviewed-by: Ben Myers <bpm@xxxxxxx> Applied. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH v3] xfs: fix memory leak in xfs_dir2_node_removename, Ben Myers |
|---|---|
| Next by Date: | hello friend did you get my proposal???, mr zh |
| Previous by Thread: | Re: [PATCH 2/2] xfs free the list of recovery items on error., Ben Myers |
| Next by Thread: | Re: [PATCH v3] xfs: fix memory leak in xfs_dir2_node_removename, Ben Myers |
| Indexes: | [Date] [Thread] [Top] [All Lists] |