Message ID | 1420529017-30472-1-git-send-email-yegorslists@googlemail.com |
---|---|
State | Rejected |
Headers | show |
On 2015-01-06 08:23, yegorslists@googlemail.com wrote: > From: Yegor Yefremov <yegorslists@googlemail.com> > > This error was reported by cppcheck. Compile tested only. > > Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> > --- > list.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/list.c b/list.c > index 0d00f81..46f8943 100644 > --- a/list.c > +++ b/list.c > @@ -346,7 +346,6 @@ uci_lookup_ext_section(struct uci_context *ctx, struct uci_ptr *ptr) > goto done; > > error: > - free(section); > memset(ptr, 0, sizeof(struct uci_ptr)); > UCI_THROW(ctx, UCI_ERR_INVAL); False positive - the "done" part is never reached due to the UCI_THROW call. - Felix
On Tue, Jan 6, 2015 at 12:12 PM, Felix Fietkau <nbd@openwrt.org> wrote: > On 2015-01-06 08:23, yegorslists@googlemail.com wrote: >> From: Yegor Yefremov <yegorslists@googlemail.com> >> >> This error was reported by cppcheck. Compile tested only. >> >> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> >> --- >> list.c | 1 - >> 1 files changed, 0 insertions(+), 1 deletions(-) >> >> diff --git a/list.c b/list.c >> index 0d00f81..46f8943 100644 >> --- a/list.c >> +++ b/list.c >> @@ -346,7 +346,6 @@ uci_lookup_ext_section(struct uci_context *ctx, struct uci_ptr *ptr) >> goto done; >> >> error: >> - free(section); >> memset(ptr, 0, sizeof(struct uci_ptr)); >> UCI_THROW(ctx, UCI_ERR_INVAL); > False positive - the "done" part is never reached due to the UCI_THROW call. OK. Compiler throws following warning for file.c because of deprecated mktemp usage file.c:739: warning: the use of `mktemp' is dangerous, better use `mkstemp' Is this worth fixing? Yegor
diff --git a/list.c b/list.c index 0d00f81..46f8943 100644 --- a/list.c +++ b/list.c @@ -346,7 +346,6 @@ uci_lookup_ext_section(struct uci_context *ctx, struct uci_ptr *ptr) goto done; error: - free(section); memset(ptr, 0, sizeof(struct uci_ptr)); UCI_THROW(ctx, UCI_ERR_INVAL); done: