LGTM. Diego Elio Pettenò — Flameeyes [email protected] — http://blog.flameeyes.eu/
On Wed, Sep 4, 2013 at 11:43 AM, Diego Biurrun <[email protected]> wrote: > --- > > Now w/o stupid and obvious mistakes. > > libavutil/mem.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavutil/mem.c b/libavutil/mem.c > index b4a190a..5f64f56 100644 > --- a/libavutil/mem.c > +++ b/libavutil/mem.c > @@ -138,7 +138,7 @@ void *av_realloc(void *ptr, size_t size) > > void *av_realloc_array(void *ptr, size_t nmemb, size_t size) > { > - if (size <= 0 || nmemb >= INT_MAX / size) > + if (!size || nmemb >= INT_MAX / size) > return NULL; > return av_realloc(ptr, nmemb * size); > } > @@ -147,9 +147,9 @@ int av_reallocp_array(void *ptr, size_t nmemb, size_t > size) > { > void **ptrptr = ptr; > void *ret; > - if (size <= 0 || nmemb >= INT_MAX / size) > + if (!size || nmemb >= INT_MAX / size) > return AVERROR(ENOMEM); > - if (nmemb <= 0) { > + if (!nmemb) { > av_freep(ptr); > return 0; > } > -- > 1.7.9.5 > > _______________________________________________ > libav-devel mailing list > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel > _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
