On Fri, May 5, 2017 at 7:49 AM, Matias Elo <[email protected]> wrote:
> odp_shm_lookup() return value can be used to detect name conflicts. > > Signed-off-by: Matias Elo <[email protected]> > Reviewed-by: Bill Fischofer <[email protected]> > --- > helper/hashtable.c | 3 +-- > helper/lineartable.c | 3 +-- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/helper/hashtable.c b/helper/hashtable.c > index 9d41144..f26b18b 100644 > --- a/helper/hashtable.c > +++ b/helper/hashtable.c > @@ -76,8 +76,7 @@ odph_table_t odph_hash_table_create(const char *name, > uint32_t capacity, > ODPH_DBG("create para input error!\n"); > return NULL; > } > - tbl = (odph_hash_table_imp *)odp_shm_addr(odp_shm_lookup(name)); > - if (tbl != NULL) { > + if (odp_shm_lookup(name) != ODP_SHM_INVALID) { > ODPH_DBG("name already exist\n"); > return NULL; > } > diff --git a/helper/lineartable.c b/helper/lineartable.c > index 32c4956..dd4a599 100644 > --- a/helper/lineartable.c > +++ b/helper/lineartable.c > @@ -55,8 +55,7 @@ odph_table_t odph_linear_table_create(const char *name, > uint32_t capacity, > return NULL; > } > /* check name confict in shm*/ > - tbl = (odph_linear_table_imp *)odp_shm_addr(odp_shm_lookup(name)); > - if (tbl != NULL) { > + if (odp_shm_lookup(name) != ODP_SHM_INVALID) { > ODPH_DBG("name already exist\n"); > return NULL; > } > -- > 2.7.4 > >
