From patchwork Wed Jul 19 11:03:30 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Menglong Dong X-Patchwork-Id: 122559 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c923:0:b0:3e4:2afc:c1 with SMTP id j3csp2371570vqt; Wed, 19 Jul 2023 04:37:57 -0700 (PDT) X-Google-Smtp-Source: APBJJlEMcoRctXXjIpDHVv3aGNzVbjpBaW5oEUbngrHJejJU6/GAlRwWiIJj4r8GmR+XWPWg2wa4 X-Received: by 2002:a05:6a20:394d:b0:134:6aff:ee6c with SMTP id r13-20020a056a20394d00b001346affee6cmr2178851pzg.41.1689766677352; Wed, 19 Jul 2023 04:37:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689766677; cv=none; d=google.com; s=arc-20160816; b=pBHlN7R/lLV+HC/x1CIK/sZVI1ZuvtaHkt8asRE2lol7J/G9jfDWIWy/5XKIx8p7Gt uhEWRgUbrESGmG4XEZgZMt9p+wbHk6MiXhkUCRLoGJC3hCluVwQk3M6lj8h36vFMvNC4 jdfdOFCabqzNNhSxcYChifgaHVWpGbrEumdd6VWyR2PlIUl4fxWRYp8B4VLRD8TUJnyt X6G0udbdK6x39R5mZdOomk8Ns3x/RJFJTRATWuui1YotN0D4T7IeF/ZuFVzUv5jyBHMx LAkPNLpKII1uriCKrREpHErlhKmnpPnntifI4EhyHWDZjI2YAaFhVPIzjM9ZZ7cfELOP mJvA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=hFsE/K6gUAIXkP05EqNENfRpK1bhpEZsF99N61vWloE=; fh=YGXs84cjjFpLmVMyjghshUqRyJrDwfqiGBUqUdcHg0g=; b=Yftx9l4E1vW6bRqV9cY9AQDlPjF0NPkDsPc7K1R850UoG3G8JKmguI7Keqdg0RRuot ncn9YsZ5te5OGlc01rQXOj/CEAi4wiX1WHMODvkx+IYNgepQ6bWl8BypBZELy43bsj4V QF/JXkSI+1njZWLpHcz3TTlHNYZOdxVHalg0X+Qe2csCARu09ig0IptFrBOQAuomwNHs KZPWTGuYwsYjtgAQXqXQqAx0fTeQuuNvCY6oydhBI3JhLpZok8FkvxYwbhc93XLg8QVo MlJ/BoxZfw2kbquseUADyEJ1GUBimOUWParGXJLtVatX79P6rlqWNYb7kdGv5wsRzBM1 Z53w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=g23qWW+P; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id fc38-20020a056a002e2600b006650ccb39e9si3439355pfb.294.2023.07.19.04.37.44; Wed, 19 Jul 2023 04:37:57 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=g23qWW+P; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229919AbjGSLFd (ORCPT + 99 others); Wed, 19 Jul 2023 07:05:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41030 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230107AbjGSLFb (ORCPT ); Wed, 19 Jul 2023 07:05:31 -0400 Received: from mail-pf1-x442.google.com (mail-pf1-x442.google.com [IPv6:2607:f8b0:4864:20::442]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E9065197; Wed, 19 Jul 2023 04:05:29 -0700 (PDT) Received: by mail-pf1-x442.google.com with SMTP id d2e1a72fcca58-668730696a4so4582570b3a.1; Wed, 19 Jul 2023 04:05:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1689764729; x=1690369529; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=hFsE/K6gUAIXkP05EqNENfRpK1bhpEZsF99N61vWloE=; b=g23qWW+P9F4/xy1Kafm+XGVUvORdC8zJAIToX89ML26LZ6ptcANrUBMFVhtQ+2c5Ch B5CV9Lx3rx2c98ImyBDrDMJbuMqPD0KqeXlNw/eT+qLAvfGyonOvV8BAg5AgrKniDesd sPw6qi06EYniqagV0wDNOiKib5lZsCHFfVHTWZ5/mIkVIblDEIuQu6tXeUT1dG1q/55x b7bSeDuYiIuBlontTBRmxHhLfmX2VtTLvhgNJqxIIVVVIQHu6gHlsGYiA+ySOFb8LxP6 CgmxC9pJ/Ir8Ji5PG6CVsavg5zsNIq6M2t/psKRf1nTMYm/xHgjC3FdMPopzgd88+eSX TEnA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689764729; x=1690369529; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=hFsE/K6gUAIXkP05EqNENfRpK1bhpEZsF99N61vWloE=; b=W2WuorzFYQzsCvbGycKDzluqEx88zGTpawFlfrLwHiB5QWJsax0mYsJj/mF1GaKiw2 i9ZITlWP7oWjErJSwZQciMxVHbtOCWwVUFdFpcEaOXQCbzu7h87xNb757PYjtiwGgT8B jxkFjHc7qWKxnjVIJkRkhQeTogK2S0LWKdFrCeP0TgD4eKpIbizQ1QxHhwXXGJH6HHcw fQ1pJFHFFMRyBXu7pm51c2YrD2xzUUnZIjrfhyFtmd4PLyeAVUUybBtP3njP/ODvisUv GMfnvll0fbbqW3eM+Y/A7VcLOiVjYriCBsRgxiFKBSg9uM7fevxGYbVWoaEjsKMQU7ix g8NQ== X-Gm-Message-State: ABy/qLbX5E+i4gNjnbiw3c/EXfiOS0PRanLF7fo9Hc6YcFoM8NlO4BRb zzI3MVgFVwQNkdCiDyBoby+BtK/3wuw4BO6vSdw= X-Received: by 2002:a05:6a21:6d89:b0:137:48cc:9cfa with SMTP id wl9-20020a056a216d8900b0013748cc9cfamr1033460pzb.24.1689764729250; Wed, 19 Jul 2023 04:05:29 -0700 (PDT) Received: from localhost.localdomain ([203.205.141.81]) by smtp.gmail.com with ESMTPSA id v10-20020a62ac0a000000b00682c864f35bsm3124196pfe.140.2023.07.19.04.05.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Jul 2023 04:05:28 -0700 (PDT) From: menglong8.dong@gmail.com X-Google-Original-From: imagedong@tencent.com To: yhs@fb.com Cc: davem@davemloft.net, dsahern@kernel.org, ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, martin.lau@linux.dev, song@kernel.org, john.fastabend@gmail.com, kpsingh@kernel.org, sdf@google.com, haoluo@google.com, jolsa@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, imagedong@tencent.com, netdev@vger.kernel.org, bpf@vger.kernel.org, linux-kernel@vger.kernel.org, Hao Peng , Dan Carpenter Subject: [PATCH bpf-next] bpf, x86: initialize the variable "first_off" in save_args() Date: Wed, 19 Jul 2023 19:03:30 +0800 Message-Id: <20230719110330.2007949-1-imagedong@tencent.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771848783354307772 X-GMAIL-MSGID: 1771848783354307772 From: Menglong Dong As Dan Carpenter reported, the variable "first_off" which is passed to clean_stack_garbage() in save_args() can be uninitialized, which can cause runtime warnings with KMEMsan. Therefore, init it with 0. Fixes: 473e3150e30a ("bpf, x86: allow function arguments up to 12 for TRACING") Cc: Hao Peng Reported-by: Dan Carpenter Closes: https://lore.kernel.org/bpf/09784025-a812-493f-9829-5e26c8691e07@moroto.mountain/ Signed-off-by: Menglong Dong --- arch/x86/net/bpf_jit_comp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/net/bpf_jit_comp.c b/arch/x86/net/bpf_jit_comp.c index 5ab531be56ac..83c4b45dc65f 100644 --- a/arch/x86/net/bpf_jit_comp.c +++ b/arch/x86/net/bpf_jit_comp.c @@ -1925,7 +1925,7 @@ static int get_nr_used_regs(const struct btf_func_model *m) static void save_args(const struct btf_func_model *m, u8 **prog, int stack_size, bool for_call_origin) { - int arg_regs, first_off, nr_regs = 0, nr_stack_slots = 0; + int arg_regs, first_off = 0, nr_regs = 0, nr_stack_slots = 0; int i, j; /* Store function arguments to stack.