>
> Dear Luigi 'Comio' Mantellini,
>
> In message <1291469358-25023-3-git-send-email-luigi.mantell...@idf-hit.com> 
> you wrote:
> >
> > Signed-off-by: Luigi 'Comio' Mantellini <luigi.mantell...@idf-hit.com>
> > ---
> >  lib/lzma/LzmaDec.c |    4 +++-
> >  1 files changed, 3 insertions(+), 1 deletions(-)
> >
> > diff --git a/lib/lzma/LzmaDec.c b/lib/lzma/LzmaDec.c
> > index f941da2..b2a3aec 100644
> > --- a/lib/lzma/LzmaDec.c
> > +++ b/lib/lzma/LzmaDec.c
> > @@ -913,7 +913,9 @@ SRes LzmaDec_DecodeToBuf(CLzmaDec *p, Byte *dest, SizeT 
> > *destLen, const Byte *sr
> >
> >  void LzmaDec_FreeProbs(CLzmaDec *p, ISzAlloc *alloc)
> >  {
> > -  alloc->Free(alloc, p->probs);
> > +  if (p->probs) {
> > +    alloc->Free(alloc, p->probs);
> > +  }
> >    p->probs = 0;
> >  }
>
> Incorrect indentation, and no braces needed for single line
> statements.

.. and that NULL test belongs in alloc->Free like the standard free() does.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to