mbox series

[OpenWrt-Devel,rpcd,v2,0/6] memory issue fixes

Message ID 20191021061031.81230-1-yszhou4tech@gmail.com
Headers show
Series memory issue fixes | expand

Message

Yousong Zhou Oct. 21, 2019, 6:10 a.m. UTC 
v2 <- v1

 - Added two fixes suggested by scan-build result [1]

[1] https://ynezz.gitlab.io/-/openwrt-rpcd/-/jobs/323420230/artifacts/build/scan/2019-10-17-075741-163-1/index.html 

Yousong Zhou (6):
  plugin: exec: properly free memory on parse error
  exec: properly free memory on rpc_exec() error
  exec: alway call finish_cb to allow plugin to free up memory
  uci: free configs list memory on return
  file: exec: properly free memory on error
  main: exec_self: make clang analyzer happy

 exec.c   | 18 +++++++++++-------
 file.c   | 18 +++++++++++++++---
 main.c   | 10 +++++++---
 plugin.c | 13 ++++++++++---
 uci.c    |  2 ++
 5 files changed, 45 insertions(+), 16 deletions(-)

Comments

Petr Štetiar Oct. 21, 2019, 10:44 a.m. UTC | #1 
Yousong Zhou <yszhou4tech@gmail.com> [2019-10-21 06:10:25]:

Hi,

> Yousong Zhou (6):

seems like two patches didn't made it to the mailing list and/or patchwork.

-- ynezz
Yousong Zhou Oct. 21, 2019, 1:01 p.m. UTC | #2 
On Mon, 21 Oct 2019 at 18:44, Petr Štetiar <ynezz@true.cz> wrote:
>
> Yousong Zhou <yszhou4tech@gmail.com> [2019-10-21 06:10:25]:
>
> Hi,
>
> > Yousong Zhou (6):
>
> seems like two patches didn't made it to the mailing list and/or patchwork.
>
> -- ynezz

Oops..  Resent the two patches as reply to this cover letter.

Regards,
                yousong
Jo-Philipp Wich Oct. 29, 2019, 8:38 a.m. UTC | #3 
Hi Yousong,

thank you for the series. I applied it with slight whitespace changes
after some valgrind testing on-target.

~ Jo