diff mbox series

[FFmpeg-devel] avcodec/evc_ps: Replace av_malloc+memset by av_mallocz

Message ID GV1P250MB0737E2E23E3F74EAB6AF96428F25A@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM
State Accepted
Commit 915701939ca5d0a801647f7916394f33a2645d83
Headers show
Series [FFmpeg-devel] avcodec/evc_ps: Replace av_malloc+memset by av_mallocz | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andreas Rheinhardt June 29, 2023, 10:28 p.m. UTC 
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/evc_ps.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

Comments

James Almer June 29, 2023, 10:46 p.m. UTC | #1 
On 6/29/2023 7:28 PM, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>   libavcodec/evc_ps.c | 8 ++------
>   1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/libavcodec/evc_ps.c b/libavcodec/evc_ps.c
> index ec405345ae..fc2105b352 100644
> --- a/libavcodec/evc_ps.c
> +++ b/libavcodec/evc_ps.c
> @@ -143,12 +143,10 @@ int ff_evc_parse_sps(GetBitContext *gb, EVCParamSets *ps)
>       if (sps_seq_parameter_set_id >= EVC_MAX_SPS_COUNT)
>           return AVERROR_INVALIDDATA;
>   
> -    sps = av_malloc(sizeof(*sps));
> +    sps = av_mallocz(sizeof(*sps));
>       if (!sps)
>           return AVERROR(ENOMEM);
>   
> -    memset(sps, 0, sizeof(*sps));
> -
>       sps->sps_seq_parameter_set_id = sps_seq_parameter_set_id;
>   
>       // the Baseline profile is indicated by profile_idc eqal to 0
> @@ -318,12 +316,10 @@ int ff_evc_parse_pps(GetBitContext *gb, EVCParamSets *ps)
>       if (pps_pic_parameter_set_id >= EVC_MAX_PPS_COUNT)
>           return AVERROR_INVALIDDATA;
>   
> -    pps = av_malloc(sizeof(*pps));
> +    pps = av_mallocz(sizeof(*pps));
>       if (!pps)
>           return AVERROR(ENOMEM);
>   
> -    memset(pps, 0, sizeof(*pps));
> -
>       pps->pps_pic_parameter_set_id = pps_pic_parameter_set_id;
>   
>       pps->pps_seq_parameter_set_id = get_ue_golomb(gb);

LGTM.
diff mbox series

Patch

diff --git a/libavcodec/evc_ps.c b/libavcodec/evc_ps.c
index ec405345ae..fc2105b352 100644
--- a/libavcodec/evc_ps.c
+++ b/libavcodec/evc_ps.c
@@ -143,12 +143,10 @@  int ff_evc_parse_sps(GetBitContext *gb, EVCParamSets *ps)
     if (sps_seq_parameter_set_id >= EVC_MAX_SPS_COUNT)
         return AVERROR_INVALIDDATA;
 
-    sps = av_malloc(sizeof(*sps));
+    sps = av_mallocz(sizeof(*sps));
     if (!sps)
         return AVERROR(ENOMEM);
 
-    memset(sps, 0, sizeof(*sps));
-
     sps->sps_seq_parameter_set_id = sps_seq_parameter_set_id;
 
     // the Baseline profile is indicated by profile_idc eqal to 0
@@ -318,12 +316,10 @@  int ff_evc_parse_pps(GetBitContext *gb, EVCParamSets *ps)
     if (pps_pic_parameter_set_id >= EVC_MAX_PPS_COUNT)
         return AVERROR_INVALIDDATA;
 
-    pps = av_malloc(sizeof(*pps));
+    pps = av_mallocz(sizeof(*pps));
     if (!pps)
         return AVERROR(ENOMEM);
 
-    memset(pps, 0, sizeof(*pps));
-
     pps->pps_pic_parameter_set_id = pps_pic_parameter_set_id;
 
     pps->pps_seq_parameter_set_id = get_ue_golomb(gb);