From 0e659094abf0bfe32e516c41e449635cd320107f Mon Sep 17 00:00:00 2001 From: Corey Minyard Date: Fri, 2 Feb 2024 14:01:38 -0600 Subject: [PATCH] gensio: Remove dead code from str_to_gensio_accepter() Move the args variable to the scope where it belongs and remove the bogus free. Signed-off-by: Corey Minyard --- lib/gensio.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/gensio.c b/lib/gensio.c index ad0923f5..83a5c4b0 100644 --- a/lib/gensio.c +++ b/lib/gensio.c @@ -1578,7 +1578,6 @@ str_to_gensio_accepter(const char *str, struct gensio_accepter **accepter) { int err = GE_INVAL; - const char **args = NULL; struct registered_gensio_accepter *r; size_t len; bool retried = false; @@ -1591,6 +1590,8 @@ str_to_gensio_accepter(const char *str, str++; retry: for (r = reg_gensio_accs; r; r = r->next) { + const char **args = NULL; + len = strlen(r->name); if (strncmp(r->name, str, len) != 0 || (str[len] != ',' && str[len] != '(' && str[len])) @@ -1612,9 +1613,6 @@ str_to_gensio_accepter(const char *str, goto retry; } - if (args) - gensio_argv_free(o, args); - return err; }