To make this check strcmp is good enought. Not need to check len first and then do memory compare. --- src/shared/hfp.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/shared/hfp.c b/src/shared/hfp.c index 9a66ed2..0ce38cc 100644 --- a/src/shared/hfp.c +++ b/src/shared/hfp.c @@ -135,10 +135,7 @@ static bool match_handler_prefix(const void *a, const void *b) const struct cmd_handler *handler = a; const char *prefix = b; - if (strlen(handler->prefix) != strlen(prefix)) - return false; - - if (memcmp(handler->prefix, prefix, strlen(prefix))) + if (strcmp(handler->prefix, prefix) != 0) return false; return true; -- 1.8.4 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html