Re: [PATCH] tools: firewire: nosy-dump: fix a resource leak in main()

2017-09-26 Thread Stefan Richter
On Sep 13 Martin Kepplinger wrote: > If option_input and option_output is true two files are opened > consecutively. In case the second fopen() fails, let's fclose() > the first one before returning early. > > Signed-off-by: Martin Kepplinger > --- >

Re: [PATCH] tools: firewire: nosy-dump: fix a resource leak in main()

2017-09-26 Thread Stefan Richter
On Sep 13 Martin Kepplinger wrote: > If option_input and option_output is true two files are opened > consecutively. In case the second fopen() fails, let's fclose() > the first one before returning early. > > Signed-off-by: Martin Kepplinger > --- > tools/firewire/nosy-dump.c | 2 ++ > 1 file

[PATCH] tools: firewire: nosy-dump: fix a resource leak in main()

2017-09-13 Thread Martin Kepplinger
If option_input and option_output is true two files are opened consecutively. In case the second fopen() fails, let's fclose() the first one before returning early. Signed-off-by: Martin Kepplinger --- tools/firewire/nosy-dump.c | 2 ++ 1 file changed, 2 insertions(+) diff

[PATCH] tools: firewire: nosy-dump: fix a resource leak in main()

2017-09-13 Thread Martin Kepplinger
If option_input and option_output is true two files are opened consecutively. In case the second fopen() fails, let's fclose() the first one before returning early. Signed-off-by: Martin Kepplinger --- tools/firewire/nosy-dump.c | 2 ++ 1 file changed, 2 insertions(+) diff --git