> -----Original Message-----
> From: Ma, WenwuX <wenwux...@intel.com>
> Sent: Monday, November 25, 2024 12:18 PM
> To: ajit.khapa...@broadcom.com; somnath.ko...@broadcom.com;
> amitpraka...@marvell.com; Gujjar, Abhinandan S
> <abhinandan.guj...@intel.com>; dev@dpdk.org; sta...@dpdk.org
> Cc: Liao, TingtingX <tingtingx.l...@intel.com>; Ma, WenwuX
> <wenwux...@intel.com>
> 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: sta...@dpdk.org
>
> Signed-off-by: Wenwu Ma <wenwux...@intel.com>
> ---
> 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
