> -----Original Message-----
> From: Ma, WenwuX <[email protected]>
> Sent: Monday, November 25, 2024 12:18 PM
> To: [email protected]; [email protected];
> [email protected]; Gujjar, Abhinandan S
> <[email protected]>; [email protected]; [email protected]
> Cc: Liao, TingtingX <[email protected]>; Ma, WenwuX
> <[email protected]>
> Subject: [PATCH 2/3] eventdev: fix uninitialized variable
>
> This patch fixes the variable 'events' may be used uninitialized.
>
> Fixes: 7901eac3409a ("eventdev: add crypto adapter implementation")
> Cc: [email protected]
>
> Signed-off-by: Wenwu Ma <[email protected]>
> ---
> lib/eventdev/rte_event_crypto_adapter.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/eventdev/rte_event_crypto_adapter.c
> b/lib/eventdev/rte_event_crypto_adapter.c
> index 939ca1cad9..1ca839510b 100644
> --- a/lib/eventdev/rte_event_crypto_adapter.c
> +++ b/lib/eventdev/rte_event_crypto_adapter.c
> @@ -685,6 +685,7 @@ eca_ops_enqueue_burst(struct
> event_crypto_adapter *adapter,
> nb_ev = 0;
> retry = 0;
> nb_enqueued = 0;
> + memset(events, 0, sizeof(events));
There is a rte_memcpy() at line number 699 which is copying response
information as event into this.
So, that will initialize all the event fields. We don't require to initialize
events here.
> num = RTE_MIN(num, BATCH_SIZE);
> for (i = 0; i < num; i++) {
> struct rte_event *ev = &events[nb_ev++];
> --
> 2.34.1
