Received: by 2002:a25:e74b:0:0:0:0:0 with SMTP id e72csp1506174ybh; Thu, 23 Jul 2020 10:30:03 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzTWiZZ/NOxEGison44dcBHJ0jaRrYfmeG2wsWT1zwayVbdw97TE0FWwomf/W/OW47j8rlJ X-Received: by 2002:a17:906:4094:: with SMTP id u20mr3469988ejj.553.1595525403669; Thu, 23 Jul 2020 10:30:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1595525403; cv=none; d=google.com; s=arc-20160816; b=kONO6bzMt7IqUTUAfBUWJ0XgoYpu1RKvb7UiHH1gaFbtr7diNmorqRncv5YaXQFLaD 2Htzfl29vDpnhkpZ8aWARB6xareDdhEFZvFJ7tuB15r8M7xHzlOf0K9U+CE4RmUEnAOj C0n9lU64D5YS5gZJ91/+L5XzGH2L89CW8QPwSu7Xrj9biYXuha3cuMkfQi5Zvx4ZCyPa w8bD/hecGDSgWQ/Zu1314icNrRL7qqDMzx8Y5VkOlnSj11T1TOhN9QB20Dyd+LU71McZ 88yIzm3eB+j+tt0dv3ecUGPgmEKYaoC/a30T6DWZ1f40BvDDFUvBd8ZjkSXbtsnqX6zI uZ3Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=1iiTgOBg0bpwCP4gGFPRxHDUM8YOMlsrqMYzJRDQ9+U=; b=jBi15tVV1Eyf5TzbWwSZXiFddXi5/w0TZelxMvweuktOSCJ4JTnKBSGhkUb6l5tth1 yCBLrXlMLL/t9XeZBgDGaotPkPc5/u0zzNfpSx3IXTD+tCWb9MBEWVmnaM/oULjK8xoB HYKT5JpQkPVzZClOQjIFAjKiN7jOKvZyhYol50Q8PLmoAJqWxDvXmwu88TuqpGZOon0j s96MgYLr2OHgU2SfpFPK9lEGYH8iIOlfv4bgWA/RKF9rzYR5cQJzSDWWy5gUbYKay8pq HL4C+oRRzxCElVPjzNFlA8V7wMXDx+qpaLyDTzJu84pfRk+5LqpkG6FQxRb4IYrRyGAi /rkw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=UwCI8+k1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id c14si2612771edr.41.2020.07.23.10.29.39; Thu, 23 Jul 2020 10:30:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=UwCI8+k1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728994AbgGWR26 (ORCPT + 99 others); Thu, 23 Jul 2020 13:28:58 -0400 Received: from mail.kernel.org ([198.145.29.99]:49902 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726650AbgGWR26 (ORCPT ); Thu, 23 Jul 2020 13:28:58 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 2835A20714; Thu, 23 Jul 2020 17:28:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1595525337; bh=VkdDKS0wN9LZ9uHZh7M2Y+USkmugM8r/87cp7BNkqhQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UwCI8+k1hDpdbuZUG5lr/L4L8Udn9SrtdS6b+mPP3r0OqMl7mhQcdycrgMaBSrquz CB9HnfsRx0WsnmEnLFc9pBYcKxB2s0VNqQPCBnPUtMxRpnFODnVMZPja7zh0X2KwLw C9SjMRMyrB5WSdAVfnlLt4T7ry22VP1bYEyAPUCw= Date: Thu, 23 Jul 2020 19:29:01 +0200 From: Greg Kroah-Hartman To: "Gustavo A. R. Silva" Cc: Tomas Winkler , Alexander Usyskin , linux-kernel@vger.kernel.org, "Gustavo A . R . Silva" , Arnd Bergmann Subject: Re: [char-misc-next 1/6] mei: hbm: use sizeof of variable instead of struct type Message-ID: <20200723172901.GA2840395@kroah.com> References: <20200723145927.882743-1-tomas.winkler@intel.com> <20200723145927.882743-2-tomas.winkler@intel.com> <20200723151310.GA2809544@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 23, 2020 at 12:04:06PM -0500, Gustavo A. R. Silva wrote: > > > On 7/23/20 10:13, Greg Kroah-Hartman wrote: > > On Thu, Jul 23, 2020 at 05:59:22PM +0300, Tomas Winkler wrote: > >> There is a possibility of bug when variable type has changed but > >> corresponding struct passed to the sizeof has not. > >> > >> Cc: Gustavo A. R. Silva > >> Signed-off-by: Tomas Winkler > >> --- > >> drivers/misc/mei/hbm.c | 74 ++++++++++++++++++------------------------ > >> 1 file changed, 32 insertions(+), 42 deletions(-) > > > > This doesn't apply to my tree as I've applied Gustavo's patch. Should I > > revert that first? > > > > Yep; this series doesn't take into account my patch. I'm OK with > reverting it, so we can apply this series. Ok, will do, thanks! greg k-h