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 X-Spam-Level: X-Spam-Status: No, score=-0.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1A07AC18E5B for ; Wed, 25 Mar 2020 09:19:35 +0000 (UTC) Received: from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id CB94F20722 for ; Wed, 25 Mar 2020 09:19:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=itdevltd.onmicrosoft.com header.i=@itdevltd.onmicrosoft.com header.b="aDp22+d0" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CB94F20722 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=itdev.co.uk Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id 857F3237C8; Wed, 25 Mar 2020 09:19:34 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Me0LZRV2wW3H; Wed, 25 Mar 2020 09:19:33 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by silver.osuosl.org (Postfix) with ESMTP id 90EFD22803; Wed, 25 Mar 2020 09:19:33 +0000 (UTC) Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id E35B81BF37C for ; Wed, 25 Mar 2020 09:19:31 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id DFAAF86CB8 for ; Wed, 25 Mar 2020 09:19:31 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id JWe67nCtNv0B for ; Wed, 25 Mar 2020 09:19:30 +0000 (UTC) X-Greylist: delayed 23:10:03 by SQLgrey-1.7.6 Received: from EUR03-VE1-obe.outbound.protection.outlook.com (mail-eopbgr50055.outbound.protection.outlook.com [40.107.5.55]) by fraxinus.osuosl.org (Postfix) with ESMTPS id 1FAD686CB2 for ; Wed, 25 Mar 2020 09:19:30 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jHQutkmedsDm43JFyDVhlgLMnXrhgFir+Bhd4q21quy8aNMq/OGadwgP6LlGbxLQpbgPgK4BOvlXXJK7MmLOOq/TaONFmaJ5aOD83Pfuudgbc+RP5i71gQfIhNhiTO2uVzznLt1u2/FbTvlGqXif+KcIj0VEzxEEhDOWMNyXpptXVJLpsuVkQtbFidzCBUQjV5MXAxQx4jAq577SHOi1XewxtqwGBtJgcEBwG3Cb81aXPv1w8bmgqS7fws3GETiivbPBeBxPDMXnurc1ulB+JlHRhBg6YVK59Lsuqdgc7RwaiuE9L/1OKWjia6AordFsi+/UqHHe7uie3fPIjjtV8Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=mkiqKGtQ4G6bCI09OepV5uXO6ATrWneKK+Ipz3Vx22Y=; b=HwOG5rQoTDn0RG2z7l4X1zMci+QQ/YWKwXMXLhGvUjBsC18clY8azxWiqHxBVQFmI635xNvpX5AhpjVmb0CeKzszvVzxai1RpmwMIhEdYfe7tEu94JWIypJKy4ye/RE4wxPP3w3Tn0xINfXkYCcJdI6Zollb7PpJ3+24lVmct9TFnedhutCiT9JZqNgXP+8hZkbQJlBZJLhGuM//iXnmQLVXY/MvhV1UBLzZYeklkfZgEbjwl5bz52zK+i6B8wWVzM1FWhgYS40lavIYDvOEypJl2UszR0gSXxVJWxJRfyEFMquz0gLn3G1B9Fp1MO/yaXKwLWCBg2xOO1OwkEaR0g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=itdev.co.uk; dmarc=pass action=none header.from=itdev.co.uk; dkim=pass header.d=itdev.co.uk; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=itdevltd.onmicrosoft.com; s=selector2-itdevltd-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=mkiqKGtQ4G6bCI09OepV5uXO6ATrWneKK+Ipz3Vx22Y=; b=aDp22+d0TmXOEVyNHuzmMcJFB7Ot2l5nAn/CxHlNZ5DIMW7wZBM1Hq0SCKCy159uWbJkd1e6B71qz3UN23soNPznMbrNYkjU5lgYTZoJvJ9lHXeGRxDHE1po+dDnucAqz9tjkhPp+1gXLkBxtOp4hBjzWID9EKhEBjHsH9N35yw= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=quentin.deslandes@itdev.co.uk; Received: from DBBPR08MB4491.eurprd08.prod.outlook.com (20.179.44.144) by DBBPR08MB4362.eurprd08.prod.outlook.com (20.179.42.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2835.18; Wed, 25 Mar 2020 09:19:27 +0000 Received: from DBBPR08MB4491.eurprd08.prod.outlook.com ([fe80::cce9:f055:f034:3659]) by DBBPR08MB4491.eurprd08.prod.outlook.com ([fe80::cce9:f055:f034:3659%4]) with mapi id 15.20.2835.023; Wed, 25 Mar 2020 09:19:26 +0000 Date: Wed, 25 Mar 2020 09:19:24 +0000 From: Quentin Deslandes To: Dan Carpenter Subject: Re: [PATCH v2] staging: vt6656: Use ARRAY_SIZE instead of hardcoded size Message-ID: <20200325091924.GB15158@jiffies> References: <20200318174015.7515-1-oscar.carter@gmx.com> <20200324095456.GA7693@jiffies> <20200324131830.GD4672@kadam> Content-Disposition: inline In-Reply-To: <20200324131830.GD4672@kadam> X-ClientProxiedBy: CWLP265CA0361.GBRP265.PROD.OUTLOOK.COM (2603:10a6:401:5e::13) To DBBPR08MB4491.eurprd08.prod.outlook.com (2603:10a6:10:d2::16) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from jiffies (5.151.93.48) by CWLP265CA0361.GBRP265.PROD.OUTLOOK.COM (2603:10a6:401:5e::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2856.18 via Frontend Transport; Wed, 25 Mar 2020 09:19:26 +0000 X-Originating-IP: [5.151.93.48] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: ca06baf0-51bc-4692-72fd-08d7d09d9d82 X-MS-TrafficTypeDiagnostic: DBBPR08MB4362: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-Forefront-PRVS: 0353563E2B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(136003)(39830400003)(346002)(396003)(376002)(366004)(33656002)(186003)(2906002)(508600001)(1076003)(956004)(26005)(9576002)(6916009)(81156014)(16526019)(81166006)(9686003)(55016002)(8676002)(44832011)(316002)(86362001)(66476007)(54906003)(4744005)(6496006)(52116002)(66556008)(53546011)(66946007)(8936002)(4326008)(5660300002)(33716001); DIR:OUT; SFP:1101; SCL:1; SRVR:DBBPR08MB4362; H:DBBPR08MB4491.eurprd08.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; Received-SPF: None (protection.outlook.com: itdev.co.uk does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 1+smiLD+mH8ggHQ0c8+3wkgs+p9qQa0DGEIEuaMRAo+a5b4JZxoTsj+kg1leVd+vgnOzKrXrc757O5DNNNeEDlgyuKXEYXKE/c68Gqbxb7veJLwpKDnlLeHhmkMThd4GxIE3ucs9PyLNkhvzTLpztPA2BN7xWOtS+VGMgACbEmLZd1Il0rvT3Xg7K/prLA91pUp/Ts8Q79cpG7RZSAC/5uWCmph0vvaZxfX7IqRhtHsoXEmPSnUhDc1J6k4hQ279s3BeLe89BnvH6vfltt/q02bhce+N9SnrE9kbz9EYvIpoyqgH0S/ydMXPpjEz5AWsUGEXaVzOlH6T1Pe3NQX35IWOBq/llUAjYNI4wpmWhau5kILLH1Xh2Fke3eW+4I8Sh8PqLJ36c7ofqF9iSGPRdtAJ1nxcI0c8G9mIfjosL/S9l6agL25/PbHjiZh+Q5KS X-MS-Exchange-AntiSpam-MessageData: R9hAToOI884s0MOABNE87qEJMvziYB21SnyqkG0bsTlNptwb976gOT6DHH2wdLH69QhSOuk178tDtOWzGrJbQKRs5zYtpZQoUD00QwGHUR+w4SoSJZ0JYcql938L37TwC8yBRuL7Q6N6oZm7zPUGqg== X-OriginatorOrg: itdev.co.uk X-MS-Exchange-CrossTenant-Network-Message-Id: ca06baf0-51bc-4692-72fd-08d7d09d9d82 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Mar 2020 09:19:26.8078 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 2d2930c4-2251-45b4-ad79-3582c5f41740 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: rJ2mypnKSfsyKH+S5DLYvBzUtUx6/rSmIdEVjRwHC8rtiNAnd0AUJkL5bXmxv9QFsS+3/PkLAQZPaU0u/rFmyCkmLnFLG2fs6bP/6eyb3Zw= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR08MB4362 X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devel@driverdev.osuosl.org, Oscar Carter , Malcolm Priestley , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Forest Bond , Gabriela Bittencourt , Colin Ian King Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" On 03/24/20 16:18:30, Dan Carpenter wrote: > That's a bit over engineering something which is pretty trivial. > Normally, we would just make the size a define instead of a magic number > 14. My bad, I meant "define", not "macro". > If people change the size in the future (unlikely) and it causes a bug > then they kind of deserve it because they need to ensure all the new > stuff is initialized, right? If they change it and it results in a > buffer overflow then static checkers would complain. If they changed it > and it resulted in uninitialized data being used then it would be zero > so that's okay. I wasn't sure where I should stand on this, that's clearer now. Thanks, Quentin _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel