Hi Gustavo, On Sat, Jun 05, 2010, Gustavo F. Padovan wrote: > Issues found by smatch static check: http://smatch.sourceforge.net/ > --- > sbc/sbc.c | 3 +-- > sbc/sbcdec.c | 9 +++------ > 2 files changed, 4 insertions(+), 8 deletions(-) > > diff --git a/sbc/sbc.c b/sbc/sbc.c > index edd152f..86399dd 100644 > --- a/sbc/sbc.c > +++ b/sbc/sbc.c > @@ -1138,8 +1138,7 @@ void sbc_finish(sbc_t *sbc) > if (!sbc) > return; > > - if (sbc->priv_alloc_base) > - free(sbc->priv_alloc_base); > + free(sbc->priv_alloc_base); I'm a bit hesitant at removing these NULL checks since older libc versions might need them. Any comments from Marcel? Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
