memcmp() can cause aborts on some platforms and generally seems to be
the wrong approach here. Use strncmp() instead which is more correct.

Signed-off-by: Caleb Connolly <caleb.conno...@linaro.org>
---
 drivers/soc/qcom/cmd-db.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/qcom/cmd-db.c b/drivers/soc/qcom/cmd-db.c
index 08736ea936ae..c7c5230983d5 100644
--- a/drivers/soc/qcom/cmd-db.c
+++ b/drivers/soc/qcom/cmd-db.c
@@ -140,9 +140,9 @@ static int cmd_db_get_header(const char *id, const struct 
entry_header **eh,
                        break;
 
                ent = rsc_to_entry_header(rsc_hdr);
                for (j = 0; j < le16_to_cpu(rsc_hdr->cnt); j++, ent++) {
-                       if (memcmp(ent->id, query, sizeof(ent->id)) == 0) {
+                       if (strncmp(ent->id, query, sizeof(ent->id)) == 0) {
                                if (eh)
                                        *eh = ent;
                                if (rh)
                                        *rh = rsc_hdr;

-- 
2.46.0

Reply via email to