Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754466AbdH1Icw (ORCPT ); Mon, 28 Aug 2017 04:32:52 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:33612 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754282AbdH1Icv (ORCPT ); Mon, 28 Aug 2017 04:32:51 -0400 Date: Mon, 28 Aug 2017 11:32:21 +0300 From: Dan Carpenter To: Joe Perches Cc: Larry Finger , Yurii Pavlenko , devel@driverdev.osuosl.org, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Staging rtlwifi efuse fix up a warning kzalloc Message-ID: <20170828083220.vvw5bxsth6yfuxb4@mwanda> References: <1503752478-9732-1-git-send-email-pyldev@gmail.com> <2ba8d35d-44e8-0509-469b-5564efe64e76@lwfinger.net> <1503761195.12569.43.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1503761195.12569.43.camel@perches.com> User-Agent: NeoMutt/20170113 (1.7.2) X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 347 Lines: 10 Yeah... I reviewed this code to see where rtlpriv->cfg->maps[EFUSE_HWSET_MAX_SIZE] gets set and if it can be controlled by the user. It looks to me like it's only set when we declare rtl8822be_hal_cfg in drivers/staging/rtlwifi/rtl8822be/sw.c so it's fine. I think it should probably be a big struct instead of an array. regards, dan carpenter