* [dpdk-dev] [PATCH] timer: fix pointer to local outside scope
@ 2019-04-24 14:33 Erik Gabriel Carrillo
2019-04-24 14:33 ` Erik Gabriel Carrillo
2019-05-03 20:57 ` Thomas Monjalon
0 siblings, 2 replies; 4+ messages in thread
From: Erik Gabriel Carrillo @ 2019-04-24 14:33 UTC (permalink / raw)
To: john.mcnamara; +Cc: marko.kovacevic, dev, stable
A null array is allowed to be passed as one of the parameters to
rte_timer_alt_manage() as a convenience. When that happened, an
anonymous array was created using compound literal syntax, and Coverity
detected that the object was out of scope in later uses of it. Create
an object in the proper scope instead.
Coverity issue: 337919
Fixes: c0749f7096c7 ("timer: allow management in shared memory")
Cc: stable@dpdk.org
Signed-off-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
---
lib/librte_timer/rte_timer.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/librte_timer/rte_timer.c b/lib/librte_timer/rte_timer.c
index eb46009..d443b8c 100644
--- a/lib/librte_timer/rte_timer.c
+++ b/lib/librte_timer/rte_timer.c
@@ -826,6 +826,7 @@ rte_timer_alt_manage(uint32_t timer_data_id,
int nb_poll_lcores,
rte_timer_alt_manage_cb_t f)
{
+ unsigned int default_poll_lcores[] = {rte_lcore_id()};
union rte_timer_status status;
struct rte_timer *tim, *next_tim, **pprev;
struct rte_timer *run_first_tims[RTE_MAX_LCORE];
@@ -847,8 +848,8 @@ rte_timer_alt_manage(uint32_t timer_data_id,
__TIMER_STAT_ADD(data->priv_timer, manage, 1);
if (poll_lcores == NULL) {
- poll_lcores = (unsigned int []){rte_lcore_id()};
- nb_poll_lcores = 1;
+ poll_lcores = default_poll_lcores;
+ nb_poll_lcores = RTE_DIM(default_poll_lcores);
}
for (i = 0; i < nb_poll_lcores; i++) {
--
2.6.4
^ permalink raw reply [flat|nested] 4+ messages in thread* [dpdk-dev] [PATCH] timer: fix pointer to local outside scope
2019-04-24 14:33 [dpdk-dev] [PATCH] timer: fix pointer to local outside scope Erik Gabriel Carrillo
@ 2019-04-24 14:33 ` Erik Gabriel Carrillo
2019-05-03 20:57 ` Thomas Monjalon
1 sibling, 0 replies; 4+ messages in thread
From: Erik Gabriel Carrillo @ 2019-04-24 14:33 UTC (permalink / raw)
To: john.mcnamara; +Cc: marko.kovacevic, dev, stable
A null array is allowed to be passed as one of the parameters to
rte_timer_alt_manage() as a convenience. When that happened, an
anonymous array was created using compound literal syntax, and Coverity
detected that the object was out of scope in later uses of it. Create
an object in the proper scope instead.
Coverity issue: 337919
Fixes: c0749f7096c7 ("timer: allow management in shared memory")
Cc: stable@dpdk.org
Signed-off-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
---
lib/librte_timer/rte_timer.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/librte_timer/rte_timer.c b/lib/librte_timer/rte_timer.c
index eb46009..d443b8c 100644
--- a/lib/librte_timer/rte_timer.c
+++ b/lib/librte_timer/rte_timer.c
@@ -826,6 +826,7 @@ rte_timer_alt_manage(uint32_t timer_data_id,
int nb_poll_lcores,
rte_timer_alt_manage_cb_t f)
{
+ unsigned int default_poll_lcores[] = {rte_lcore_id()};
union rte_timer_status status;
struct rte_timer *tim, *next_tim, **pprev;
struct rte_timer *run_first_tims[RTE_MAX_LCORE];
@@ -847,8 +848,8 @@ rte_timer_alt_manage(uint32_t timer_data_id,
__TIMER_STAT_ADD(data->priv_timer, manage, 1);
if (poll_lcores == NULL) {
- poll_lcores = (unsigned int []){rte_lcore_id()};
- nb_poll_lcores = 1;
+ poll_lcores = default_poll_lcores;
+ nb_poll_lcores = RTE_DIM(default_poll_lcores);
}
for (i = 0; i < nb_poll_lcores; i++) {
--
2.6.4
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] timer: fix pointer to local outside scope
2019-04-24 14:33 [dpdk-dev] [PATCH] timer: fix pointer to local outside scope Erik Gabriel Carrillo
2019-04-24 14:33 ` Erik Gabriel Carrillo
@ 2019-05-03 20:57 ` Thomas Monjalon
2019-05-03 20:57 ` Thomas Monjalon
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Monjalon @ 2019-05-03 20:57 UTC (permalink / raw)
To: Erik Gabriel Carrillo; +Cc: dev, john.mcnamara, marko.kovacevic, stable
24/04/2019 16:33, Erik Gabriel Carrillo:
> A null array is allowed to be passed as one of the parameters to
> rte_timer_alt_manage() as a convenience. When that happened, an
> anonymous array was created using compound literal syntax, and Coverity
> detected that the object was out of scope in later uses of it. Create
> an object in the proper scope instead.
>
> Coverity issue: 337919
> Fixes: c0749f7096c7 ("timer: allow management in shared memory")
> Cc: stable@dpdk.org
No need to Cc stable for a commit introduced in this release.
> Signed-off-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
Applied, thanks
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] timer: fix pointer to local outside scope
2019-05-03 20:57 ` Thomas Monjalon
@ 2019-05-03 20:57 ` Thomas Monjalon
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2019-05-03 20:57 UTC (permalink / raw)
To: Erik Gabriel Carrillo; +Cc: dev, john.mcnamara, marko.kovacevic, stable
24/04/2019 16:33, Erik Gabriel Carrillo:
> A null array is allowed to be passed as one of the parameters to
> rte_timer_alt_manage() as a convenience. When that happened, an
> anonymous array was created using compound literal syntax, and Coverity
> detected that the object was out of scope in later uses of it. Create
> an object in the proper scope instead.
>
> Coverity issue: 337919
> Fixes: c0749f7096c7 ("timer: allow management in shared memory")
> Cc: stable@dpdk.org
No need to Cc stable for a commit introduced in this release.
> Signed-off-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
Applied, thanks
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-05-03 20:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-24 14:33 [dpdk-dev] [PATCH] timer: fix pointer to local outside scope Erik Gabriel Carrillo
2019-04-24 14:33 ` Erik Gabriel Carrillo
2019-05-03 20:57 ` Thomas Monjalon
2019-05-03 20:57 ` Thomas Monjalon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).