From nobody Tue Jun 30 13:54:47 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 715FAC4332F for ; Sun, 16 Jan 2022 14:43:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235459AbiAPOmV (ORCPT ); Sun, 16 Jan 2022 09:42:21 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]:52873 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235447AbiAPOmT (ORCPT ); Sun, 16 Jan 2022 09:42:19 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1642344138; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=JRRT3Yl7+f6HbkWI411SEh1HSKqxqkLW8Pqe76fS2Gs=; b=c9r1HtrTQmy2v1uvmEc51TO9JD0VGcGfRwEieTTj7+/i7fW8hSvp01ZMXM3nTS/kGfxhdU dPmJWURTG5V8dFDz9Hye8ztv24Sq/YvlqUo5undOKcVRvhzNTzP5CdeoexEARaIVvqrJ+I g2m/w3pCCrCpIhCfnyTSXwDUZ6I8os0= Received: from mail-ot1-f71.google.com (mail-ot1-f71.google.com [209.85.210.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-341-3bgqf_c7PK2i0F5fw_7NSg-1; Sun, 16 Jan 2022 09:42:17 -0500 X-MC-Unique: 3bgqf_c7PK2i0F5fw_7NSg-1 Received: by mail-ot1-f71.google.com with SMTP id a89-20020a9d2662000000b005909e4d9585so4332493otb.18 for ; Sun, 16 Jan 2022 06:42:17 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=JRRT3Yl7+f6HbkWI411SEh1HSKqxqkLW8Pqe76fS2Gs=; b=xqeja6V6Jj8tHOl0qQcmRj0hFhT0uPLGiGGiKWvd1K6IUyDs1dmtbxZ+4xQj4/IQVg r5ExiTmL8x0jtTKJKObZzXO/FQjKnB0u9UPC6TuWHhhAP2twwaubK4u9AYtNXNBunS6Q aBvP7WygH0oskFwklhIkmqbhZzVExku9NeaMZ54Kq0ApuAI2DcDACnC7lFUGJPb/k44U J3y8vUIFlwssjBvqFnVpWRytJIWqwC/6bFD8+TW+1sC2DESpJA4BnIEL0hJ6iBrhglrz aI6I8kIhEFvOxXY1oWtjWrztB6latv7WdJcEu+dkvnxXCrnZF7KrrXx9kOMp9d4cozRg PLaA== X-Gm-Message-State: AOAM531KNIOOP85iXiX8Iashpjq2ncc6PbUUkojY3UNOgJI8fVdvszTm zVlbbn8r6XP7NWYPARouIBMTmWbybw4zxmKm0IJBVW6oSMacbVSzcgeyM6Nm2p2V4jJ3s0LUvsM BWn3v0NZiSLmRE9Rds602sWSR X-Received: by 2002:aca:1c16:: with SMTP id c22mr4588769oic.83.1642344136528; Sun, 16 Jan 2022 06:42:16 -0800 (PST) X-Google-Smtp-Source: ABdhPJwrmwZp7l+rQuzj7d+eeD89HQdHOy2AEvZ4lDFXM1Xjeb9/2maMuS4teFTtAREUYd6VJgPcwQ== X-Received: by 2002:aca:1c16:: with SMTP id c22mr4588753oic.83.1642344136355; Sun, 16 Jan 2022 06:42:16 -0800 (PST) Received: from localhost.localdomain.com (024-205-208-113.res.spectrum.com. [24.205.208.113]) by smtp.gmail.com with ESMTPSA id b22sm2168045otl.24.2022.01.16.06.42.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 16 Jan 2022 06:42:15 -0800 (PST) From: trix@redhat.com To: kvalo@kernel.org, davem@davemloft.net, kuba@kernel.org, nathan@kernel.org, ndesaulniers@google.com, akolli@codeaurora.org Cc: ath11k@lists.infradead.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, Tom Rix Subject: [PATCH] ath11k: fix error handling in ath11k_qmi_assign_target_mem_chunk() Date: Sun, 16 Jan 2022 06:42:06 -0800 Message-Id: <20220116144206.399385-1-trix@redhat.com> X-Mailer: git-send-email 2.26.3 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Tom Rix Clang static analysis reports this problem qmi.c:1935:5: warning: Undefined or garbage value returned to caller return ret; ^~~~~~~~~~ ret is uninitialized. When of_parse_phandle() fails, garbage is returned. So return -EINVAL. Fixes: 6ac04bdc5edb ("ath11k: Use reserved host DDR addresses from DT for P= CI devices") Signed-off-by: Tom Rix --- drivers/net/wireless/ath/ath11k/qmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath11k/qmi.c b/drivers/net/wireless/a= th/ath11k/qmi.c index 65d3c6ba35ae6..81b2304b1fdeb 100644 --- a/drivers/net/wireless/ath/ath11k/qmi.c +++ b/drivers/net/wireless/ath/ath11k/qmi.c @@ -1932,7 +1932,7 @@ static int ath11k_qmi_assign_target_mem_chunk(struct = ath11k_base *ab) if (!hremote_node) { ath11k_dbg(ab, ATH11K_DBG_QMI, "qmi fail to get hremote_node\n"); - return ret; + return -EINVAL; } =20 ret =3D of_address_to_resource(hremote_node, 0, &res); --=20 2.26.3