[dpdk-dev] [PATCH] lpm: fix allocation of an existing object
Pavan Nikhilesh
pbhagavatula at caviumnetworks.com
Tue Jan 30 19:51:54 CET 2018
Fix rte_lpm_create_*() functions to return NULL and set rte_errno to
EEXIST when lpm object name already exists.
This is the behavior described in the API documentation in the header
file.
Fixes: 134975073af3 ("lib: remove unnecessary pointer cast")
Signed-off-by: Pavan Nikhilesh <pbhagavatula at caviumnetworks.com>
---
This was already fixed in a previous patch by Olivier commit id
f82f705b635d31a63446a16bc4526dbebf293c5a.
But for some reason was undone in 134975073af3.
lpm6 is untouched.
Fixes currently broken 'func_reentrancy_autotest'.
lib/librte_lpm/rte_lpm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c
index d464dbda9..d00b13d93 100644
--- a/lib/librte_lpm/rte_lpm.c
+++ b/lib/librte_lpm/rte_lpm.c
@@ -180,6 +180,7 @@ rte_lpm_create_v20(const char *name, int socket_id, int max_rules,
}
if (te != NULL) {
+ lpm = NULL;
rte_errno = EEXIST;
goto exit;
}
@@ -256,6 +257,7 @@ rte_lpm_create_v1604(const char *name, int socket_id,
}
if (te != NULL) {
+ lpm = NULL;
rte_errno = EEXIST;
goto exit;
}
--
2.16.0
More information about the dev
mailing list