Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp621024pxj; Thu, 3 Jun 2021 15:18:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxhJcy7dFFROKKgk99zxaQgjQG7rKBha/8xO03K9A/RQJOiyLyB0PdEr8EwdGyTB4GF29kK X-Received: by 2002:a05:6402:42cc:: with SMTP id i12mr1581940edc.272.1622758699760; Thu, 03 Jun 2021 15:18:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1622758699; cv=none; d=google.com; s=arc-20160816; b=bLe0kdlnYClVhALkPdqbCoUjiLfs92JHSsHOfs/cgz+QpA3z9kndnGmI96CvwjXygt GpHd3PG1UXdnUj8O4/n/AomGgXZVQddfQccn06uar+cvRVgxBIEFvaG1GQ1kxFuhOIso OLAkM5GrzaSQx+b5VmXbmZug7zNdWqvd9NZh6/VLEnx/xquiMlcx1MRbbpLGiyPUpGae gyW/191lF0XysIs4EbG84f9kfyc2myINHVepNr1krQfyNh1OwMdhfBIrsISIdeKL2BPe IKJZgaWveSZTiKgmnbdUSQAZhyImfErhCgSwhf5ZSQ/nhhkxs3U7+FOOUtSBkQ29CEQt FwSw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=Xlh/oNiO3TZl/popVg/iq16H/seu6puB9V0a6+PnUTA=; b=jKimim1AU2VRkNeSeukxGZOAnV0Lvyns/X8q5lrj/Vg9SGVekPOh7bp4gHhu/lxBZg 46ub0zbrXlmIsF3ABvTmW0Pq/L4kfnLOf0qCrTig+2cTBJULZ77xCyQ13t0l293Akjq/ UWCKLt/BMGSPbSKcCmzKU8dJbs9QjDBNqMDRjpmRqt+b6MiyMU1I8ZrJzenqR+dQNQ9s +n7GN5vuIgE8pHJevUNl3KWGgtiWOR4dY/Os2vBnvRyi6kBLQrwPeahNeh2vaOoFvC7v 2yZviBudu2tEeIADsC4+5NgQN2/NLT3G5dVkM25NBezpi33+Q+djhP+x3UkHvPP21Y45 Vufg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=ii2flrQz; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id qo1si3147747ejb.590.2021.06.03.15.17.54; Thu, 03 Jun 2021 15:18:19 -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=@chromium.org header.s=google header.b=ii2flrQz; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230340AbhFCWRK (ORCPT + 99 others); Thu, 3 Jun 2021 18:17:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33388 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230300AbhFCWRJ (ORCPT ); Thu, 3 Jun 2021 18:17:09 -0400 Received: from mail-pf1-x42e.google.com (mail-pf1-x42e.google.com [IPv6:2607:f8b0:4864:20::42e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71604C06174A for ; Thu, 3 Jun 2021 15:15:12 -0700 (PDT) Received: by mail-pf1-x42e.google.com with SMTP id c12so6000376pfl.3 for ; Thu, 03 Jun 2021 15:15:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=Xlh/oNiO3TZl/popVg/iq16H/seu6puB9V0a6+PnUTA=; b=ii2flrQzSCfBbiKjmatRgjNHRcy4XbdREu4axA/t8CDmXw7J+p7pAHClIv19jO0kjq +scvaH0csxFb6xSmioqJuWkhfeE+B9h5NzZqFeQYUhDFPT9bVbXHUMr31H/9MaDFXX+8 oDM9ZLmBLYKXZDlu7Se7l2D+oAWPTtQP2h3kI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=Xlh/oNiO3TZl/popVg/iq16H/seu6puB9V0a6+PnUTA=; b=bZPKDPmUaf1riUW/Ed+rQAKN3zVM4tx+sCJ05/Bo0/cDhUVK01/zSMqPnDJNh2rYeE AvfykOkKYYKoFHq6kjLh3GsDlm5/++L7U2lvtwvscy9dj9UuwtDIqDC9zF1TpyTeCRpo HjMNiwVGcpd0CQrJLe2NliTuQXKPpPneTVD3mW4c4cNagZlZ/oubn7ARiiiHUlFZKrIj WouWc//C0R1cNgnpfg5ijpO9HQ8BMQKfrQnfvAS7d8wRxh8JHQySUgNcZzWzSDKkS1pP 8lJnQ1Pr1S/rrjbW5taf+mrz9jvoWEMSXj/aaAHK2ODWxJRCvjXCi7pavO6NnU2z4sBO H8Ew== X-Gm-Message-State: AOAM531eSgto+M+lpX1pg6iuyDH0zOnHz+a6XDrLm7Ep4B7ZiRQ28gQG 6uGcJVYKKf7NkHVD5SGHr/zR3w== X-Received: by 2002:a63:d754:: with SMTP id w20mr1648264pgi.5.1622758511958; Thu, 03 Jun 2021 15:15:11 -0700 (PDT) Received: from www.outflux.net (smtp.outflux.net. [198.145.64.163]) by smtp.gmail.com with ESMTPSA id j3sm66799pfe.98.2021.06.03.15.15.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 03 Jun 2021 15:15:11 -0700 (PDT) Date: Thu, 3 Jun 2021 15:15:09 -0700 From: Kees Cook To: "Gustavo A. R. Silva" Cc: Stanimir Varbanov , Andy Gross , Bjorn Andersson , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hardening@vger.kernel.org Subject: Re: [PATCH v3][venus-for-next-v5.14] media: venus: hfi_cmds: Fix packet size calculation Message-ID: <202106031514.BA827D5@keescook> References: <20210603001708.GA55239@embeddedor> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210603001708.GA55239@embeddedor> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 02, 2021 at 07:17:08PM -0500, Gustavo A. R. Silva wrote: > Now that a one-element array was replaced with a flexible-array member > in struct hfi_sys_set_property_pkt, use the struct_size() helper to > correctly calculate the packet size. > > Fixes: 701e10b3fd9f ("media: venus: hfi_cmds.h: Replace one-element array with flexible-array member") > Signed-off-by: Gustavo A. R. Silva Thanks for the update! Reviewed-by: Kees Cook -- Kees Cook