# HG changeset patch # User kostya # Date 1157802041 0 # Node ID 8765ee4eaa45f51cea0a2ddcecc5ea4bf3a766ae # Parent e9be5d4bad80d7c9233804f7fb4241ed8f8f3cd8 Drop unneeded checks before av_free() and change to av_freep() where it's more suitable. diff -r e9be5d4bad80 -r 8765ee4eaa45 kmvc.c --- a/kmvc.c Sat Sep 09 11:32:36 2006 +0000 +++ b/kmvc.c Sat Sep 09 11:40:41 2006 +0000 @@ -395,10 +395,8 @@ { KmvcContext *const c = (KmvcContext *) avctx->priv_data; - if (c->frm0) - av_free(c->frm0); - if (c->frm1) - av_free(c->frm1); + av_freep(&c->frm0); + av_freep(&c->frm1); if (c->pic.data[0]) avctx->release_buffer(avctx, &c->pic); diff -r e9be5d4bad80 -r 8765ee4eaa45 smacker.c --- a/smacker.c Sat Sep 09 11:32:36 2006 +0000 +++ b/smacker.c Sat Sep 09 11:40:41 2006 +0000 @@ -550,14 +550,10 @@ { SmackVContext * const smk = (SmackVContext *)avctx->priv_data; - if(smk->mmap_tbl) - av_free(smk->mmap_tbl); - if(smk->mclr_tbl) - av_free(smk->mclr_tbl); - if(smk->full_tbl) - av_free(smk->full_tbl); - if(smk->type_tbl) - av_free(smk->type_tbl); + av_freep(&smk->mmap_tbl); + av_freep(&smk->mclr_tbl); + av_freep(&smk->full_tbl); + av_freep(&smk->type_tbl); if (smk->pic.data[0]) avctx->release_buffer(avctx, &smk->pic); diff -r e9be5d4bad80 -r 8765ee4eaa45 vc1.c --- a/vc1.c Sat Sep 09 11:32:36 2006 +0000 +++ b/vc1.c Sat Sep 09 11:40:41 2006 +0000 @@ -4165,18 +4165,18 @@ // do parse frame header if(v->profile < PROFILE_ADVANCED) { if(vc1_parse_frame_header(v, &s->gb) == -1) { - if(buf2)av_free(buf2); + av_free(buf2); return -1; } } else { if(vc1_parse_frame_header_adv(v, &s->gb) == -1) { - if(buf2)av_free(buf2); + av_free(buf2); return -1; } } if(s->pict_type != I_TYPE && !v->res_rtm_flag){ - if(buf2)av_free(buf2); + av_free(buf2); return -1; } @@ -4186,7 +4186,7 @@ /* skip B-frames if we don't have reference frames */ if(s->last_picture_ptr==NULL && (s->pict_type==B_TYPE || s->dropable)){ - if(buf2)av_free(buf2); + av_free(buf2); return -1;//buf_size; } /* skip b frames if we are in a hurry */ @@ -4194,12 +4194,12 @@ if( (avctx->skip_frame >= AVDISCARD_NONREF && s->pict_type==B_TYPE) || (avctx->skip_frame >= AVDISCARD_NONKEY && s->pict_type!=I_TYPE) || avctx->skip_frame >= AVDISCARD_ALL) { - if(buf2)av_free(buf2); + av_free(buf2); return buf_size; } /* skip everything if we are in a hurry>=5 */ if(avctx->hurry_up>=5) { - if(buf2)av_free(buf2); + av_free(buf2); return -1;//buf_size; } @@ -4211,7 +4211,7 @@ } if(MPV_frame_start(s, avctx) < 0) { - if(buf2)av_free(buf2); + av_free(buf2); return -1; } @@ -4243,7 +4243,7 @@ /* we substract 1 because it is added on utils.c */ avctx->frame_number = s->picture_number - 1; - if(buf2)av_free(buf2); + av_free(buf2); return buf_size; } diff -r e9be5d4bad80 -r 8765ee4eaa45 zmbv.c --- a/zmbv.c Sat Sep 09 11:32:36 2006 +0000 +++ b/zmbv.c Sat Sep 09 11:40:41 2006 +0000 @@ -671,10 +671,8 @@ #ifdef CONFIG_ZLIB inflateEnd(&(c->zstream)); #endif - if(c->cur) - av_freep(&c->cur); - if(c->prev) - av_freep(&c->prev); + av_freep(&c->cur); + av_freep(&c->prev); return 0; }