From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5DC5FC7EE2D for ; Mon, 27 Feb 2023 10:08:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:Reply-To:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References:Cc:To:Subject: From:MIME-Version:Date:Message-ID:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ucpErYzWd+10lqMJ/wvwb5Yc5K1knV7apYzrHtVqBDY=; b=JKVuyoQGElELdUI1Abs2h/XPw4 neF5gm/GZxT9oSSveC6o9VRw+UIFfuAzhoakGe1/+PPODfAo+mg50Xj88qbaHnvubd1f47dMvNH8x mlinx1UAJ8imEwBEkrQNog/06O0mUVGyZopRsMofLOfkyyoHVoN1CsbBJN0PCkun6XzxdkFmabTxq vgqyLMpa9NG2zdt4JUnlU9ugV0PUVb9bTDCCAIVAEJ90zzGj7647kiRkvdOZh/G5bVMceR7XNg/1W y3w1NqM14Q75otEmsikiP3+AEwlo+uuVGtvI/oQNp+Nn4ojQ9nza/4gRgbAD/Q0tkQ0AUh27BG7Ib kCMxuumA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pWaPf-009AUN-Uc; Mon, 27 Feb 2023 10:07:20 +0000 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pWaPb-009ARp-HD for linux-arm-kernel@lists.infradead.org; Mon, 27 Feb 2023 10:07:17 +0000 Received: by mail-wr1-x436.google.com with SMTP id j2so5550089wrh.9 for ; Mon, 27 Feb 2023 02:07:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=Nsxevlo9XIw5q3mnw9u9AhqR6NTYzhuz0YftWRMvHew=; b=NDbGN3et9/mnN3haPMhulR5jj0TgVur9/HfZJLKx7RpqwVKMVvvLwPomFxtk+cWJLC cssqKGCjA8zKncQ62qTTjpCkO4OBFV+kJlkdXvXXwESMBt42loumc6GkOhQ2vClarTHc 6pHKUg3pYLsLLtjy54+/NshA76eehGsBKnADO7QQslYp3bvtYtHwzE1S/AC9BLjGcVcD qM0LhMT4Q6kyV7v9ydTy7cb2VwJLCuv+alc2xihBR2BlhG8KORpNSyRAnSztaLiJ2epq W2jY30TnN8V9jJnbrLKKrcoKxIegOuA379zGwl9gtQtykZb0vQnM9iK/nIL8O7KkDwx9 Bo3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Nsxevlo9XIw5q3mnw9u9AhqR6NTYzhuz0YftWRMvHew=; b=4PO6vgIJjJEZQx5k1n7dzVGAoYSt4YXtm7q+MqbY2y13R0sEtWaoHyuzSaIdFePl9F EcNHp+YL/SKuQ/OD2NCfo71LL1eZ9A/PbSGxBiJdq27UXBa9EKsKE5H/UWX3KDGemc42 g5l6m2330DG63GSg0170EX23jWPWBNlDPwPb0/1SYXdNH2WEYCB/aJnEnzKbzFkPf0x5 vPxh3cm+s/S7DGJQp7DB/veDn9uyYCZm2db+4jXAxz4NmYU4rgfra3AIRV2X2PYxdiml L4d6eJU/iaIu1Y0z9QTlSIMwtPDAhZerS7XGJn5HElMNJrui+3hZ1KJroQOCeSNEoH7+ AVzQ== X-Gm-Message-State: AO0yUKVynaDUV++mfT23AYK3jG20SWz16MI0zlq9wYRIamqjyevZrNi9 RRVaOLaTwLh5zyPYkkf8MrIMVw== X-Google-Smtp-Source: AK7set/wJKbd9qt2mIOgNl6IK492UP4h+NstK+X13zcw/O3pvbJuLqEkrddz0dZwyZ2kvPGMTckXhg== X-Received: by 2002:adf:dfce:0:b0:2c7:1b3e:362c with SMTP id q14-20020adfdfce000000b002c71b3e362cmr9316362wrn.42.1677492430808; Mon, 27 Feb 2023 02:07:10 -0800 (PST) Received: from ?IPV6:2a01:e0a:982:cbb0:a200:b8a1:f5df:d07e? ([2a01:e0a:982:cbb0:a200:b8a1:f5df:d07e]) by smtp.gmail.com with ESMTPSA id b10-20020adfee8a000000b002c54c8e70b1sm6908585wro.9.2023.02.27.02.07.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 27 Feb 2023 02:07:10 -0800 (PST) Message-ID: Date: Mon, 27 Feb 2023 11:07:08 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 From: Neil Armstrong Subject: Re: [PATCH v3] mtd: rawnand: meson: initialize struct with zeroes Content-Language: en-US To: Arseniy Krasnov , Liang Yang , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Kevin Hilman , Jerome Brunet , Martin Blumenstingl Cc: linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@sberdevices.ru, oxffffaa@gmail.com References: <20230227095906.754178-1-AVKrasnov@sberdevices.ru> Organization: Linaro Developer Services In-Reply-To: <20230227095906.754178-1-AVKrasnov@sberdevices.ru> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230227_020715_778290_5960A6F8 X-CRM114-Status: GOOD ( 19.37 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: neil.armstrong@linaro.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 27/02/2023 10:59, Arseniy Krasnov wrote: > This structure must be zeroed, because it's field 'hw->core' is used as > 'parent' in 'clk_core_fill_parent_index()', but it will be uninitialized. > This happens, because when this struct is not zeroed, pointer 'hw' is > "initialized" by garbage, which is valid pointer, but points to some > garbage. So 'hw' will be dereferenced, but 'core' contains some random > data which will be interpreted as a pointer. The following backtrace is > result of dereference of such pointer: > > [ 1.081319] __clk_register+0x414/0x820 > [ 1.085113] devm_clk_register+0x64/0xd0 > [ 1.088995] meson_nfc_probe+0x258/0x6ec > [ 1.092875] platform_probe+0x70/0xf0 > [ 1.096498] really_probe+0xc8/0x3e0 > [ 1.100034] __driver_probe_device+0x84/0x190 > [ 1.104346] driver_probe_device+0x44/0x120 > [ 1.108487] __driver_attach+0xb4/0x220 > [ 1.112282] bus_for_each_dev+0x78/0xd0 > [ 1.116077] driver_attach+0x2c/0x40 > [ 1.119613] bus_add_driver+0x184/0x240 > [ 1.123408] driver_register+0x80/0x140 > [ 1.127203] __platform_driver_register+0x30/0x40 > [ 1.131860] meson_nfc_driver_init+0x24/0x30 > > Fixes: 1e4d3ba66888 ("mtd: rawnand: meson: fix the clock") > Signed-off-by: Arseniy Krasnov > Acked-by: Martin Blumenstingl > --- > drivers/mtd/nand/raw/meson_nand.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/nand/raw/meson_nand.c b/drivers/mtd/nand/raw/meson_nand.c > index 5ee01231ac4c..30e326adabfc 100644 > --- a/drivers/mtd/nand/raw/meson_nand.c > +++ b/drivers/mtd/nand/raw/meson_nand.c > @@ -991,7 +991,7 @@ static const struct mtd_ooblayout_ops meson_ooblayout_ops = { > > static int meson_nfc_clk_init(struct meson_nfc *nfc) > { > - struct clk_parent_data nfc_divider_parent_data[1]; > + struct clk_parent_data nfc_divider_parent_data[1] = {0}; > struct clk_init_data init = {0}; > int ret; > Reviewed-by: Neil Armstrong _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel