Re: [U-Boot] [PATCH v2 1/3] dfu: Fix data abort in dfu_free_entities()

2018-07-24 Thread Lukasz Majewski
Hi Sam, > On Fri, Jul 13, 2018 at 4:35 PM, Sam Protsenko > wrote: > > Commit 5d8fae79163e ("dfu: avoid memory leak") brings a regression > > which described below. This patch is effectively reverting that > > commit, adding corresponding comment to avoid such regressions in > > future. > > > >

Re: [U-Boot] [PATCH v2 1/3] dfu: Fix data abort in dfu_free_entities()

2018-07-24 Thread Sam Protsenko
On Fri, Jul 13, 2018 at 4:35 PM, Sam Protsenko wrote: > Commit 5d8fae79163e ("dfu: avoid memory leak") brings a regression which > described below. This patch is effectively reverting that commit, adding > corresponding comment to avoid such regressions in future. > > In case of error in

[U-Boot] [PATCH v2 1/3] dfu: Fix data abort in dfu_free_entities()

2018-07-13 Thread Sam Protsenko
Commit 5d8fae79163e ("dfu: avoid memory leak") brings a regression which described below. This patch is effectively reverting that commit, adding corresponding comment to avoid such regressions in future. In case of error in dfu_config_entities(), it frees "dfu" array, which leads to "data abort"