Messages in this thread Patch in this message | | | From | Shaun Ruffell <> | Subject | [PATCH 1/3] edac_mc: fix kfree calls in the error path | Date | Sat, 18 Aug 2012 23:11:22 -0500 |
| |
From: Fengguang Wu <fengguang.wu@intel.com>
We need to free up memory in this order:
free csrows[i]->channels[j] free csrows[i]->channels free csrows[i] free csrows
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> --- drivers/edac/edac_mc.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c index 616d90b..9037ffa 100644 --- a/drivers/edac/edac_mc.c +++ b/drivers/edac/edac_mc.c @@ -419,14 +419,16 @@ error: kfree(mci->dimms); } if (mci->csrows) { - for (chn = 0; chn < tot_channels; chn++) { - csr = mci->csrows[chn]; + for (row = 0; row < tot_csrows; row++) { + csr = mci->csrows[row]; if (csr) { - for (chn = 0; chn < tot_channels; chn++) - kfree(csr->channels[chn]); + if (csr->channels) { + for (chn = 0; chn < tot_channels; chn++) + kfree(csr->channels[chn]); + kfree(csr->channels); + } kfree(csr); } - kfree(mci->csrows[i]); } kfree(mci->csrows); } -- 1.7.11.2
| |