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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 27AA5C433FE for ; Fri, 11 Nov 2022 19:22:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232004AbiKKTVx (ORCPT ); Fri, 11 Nov 2022 14:21:53 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59484 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231261AbiKKTVx (ORCPT ); Fri, 11 Nov 2022 14:21:53 -0500 Received: from mail-pl1-x62a.google.com (mail-pl1-x62a.google.com [IPv6:2607:f8b0:4864:20::62a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7FD9111464 for ; Fri, 11 Nov 2022 11:21:51 -0800 (PST) Received: by mail-pl1-x62a.google.com with SMTP id b21so4934170plc.9 for ; Fri, 11 Nov 2022 11:21:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=IHk/WyAgMPc33IOyOTaPaWGnmZhp17CsZbK+KP95Wh8=; b=O6YivlkQdE2NV4Z4tLE02IOQ7YQgIRJrWM80tdZkt5k+XIQtZz7cxKpCMaB7rrfPq7 G3XacIpYr2nn6HNsDJtbm6DA3HrV4P+qQd/qmV5UhkMSi3baY1lPJVMFkAstEJzuHUeN DgKOEOliQUckGTg21sefxcOucamEjp7kTw/Qs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=IHk/WyAgMPc33IOyOTaPaWGnmZhp17CsZbK+KP95Wh8=; b=ELXyZdr7s2PePEZKHAZ79iJ13Yi5GTp1o7lduMn5FzcR1MNNtiR2vmvhbbxmVy7QLp 2MRtG5Fk/3mI2L2SEtCoCDgGLeAIFgnuBq5o7W0jvpZS7aT9z0608ZVQXM1uMxc+6Ze2 uF0gvNRVVRZ9NDm4viBVTo8ldfEk0V1otRMtwy0SAQHuKi+yMssq2T1evGc2bxUkfnPr +QwPOn8VnA0Ek3qszD/vVGw3mO9Lf3Bhh2Yk4eTOXS3TcoemB9qVMyz9Au+Iy1oEfVAV /UaAy8mQhHWLa3XzyQQep21BOgNhobRFNVfm6VXKFaGmQqxbZSA0FuneE7aWzaJQ3wwU 6tOQ== X-Gm-Message-State: ANoB5plQL5vPvpcMh4AKXke+9B82nxaviQi1fqecgKky88YeoNSBUJkR pKERSLjTTf9zmspQSjR09Dq/fw== X-Google-Smtp-Source: AA0mqf4i77VIhs5qpNyBlVvhpR2s9r/2VBj54xZbpPIy/ryD5gcsls//pP1BrhZB4gM/fwDn5Bz05Q== X-Received: by 2002:a17:902:e052:b0:172:f5b1:e73b with SMTP id x18-20020a170902e05200b00172f5b1e73bmr3962831plx.58.1668194510942; Fri, 11 Nov 2022 11:21:50 -0800 (PST) Received: from www.outflux.net (smtp.outflux.net. [198.145.64.163]) by smtp.gmail.com with ESMTPSA id c14-20020a62f84e000000b0052d4cb47339sm1907746pfm.151.2022.11.11.11.21.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 11 Nov 2022 11:21:50 -0800 (PST) Date: Fri, 11 Nov 2022 11:21:49 -0800 From: Kees Cook To: Jani Nikula Cc: Lyude Paul , Thomas Zimmermann , linux-kernel@vger.kernel.org, Ilia Mirkin , Daniel Vetter , Karol Herbst , Ben Skeggs , David Airlie , "Nathan E. Egge" , dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org, Dave Airlie , "Gustavo A. R. Silva" , linux-next@vger.kernel.org, linux-hardening@vger.kernel.org Subject: Re: Coverity: nouveau_dp_irq(): Null pointer dereferences Message-ID: <202211111110.11B554B@keescook> References: <202211100850.7A8DD75@keescook> <87iljl6ehe.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87iljl6ehe.fsf@intel.com> Precedence: bulk List-ID: X-Mailing-List: linux-next@vger.kernel.org On Fri, Nov 11, 2022 at 11:13:17AM +0200, Jani Nikula wrote: > On Thu, 10 Nov 2022, coverity-bot wrote: > > Hello! > > > > This is an experimental semi-automated report about issues detected by > > Coverity from a scan of next-20221110 as part of the linux-next scan project: > > https://scan.coverity.com/projects/linux-next-weekly-scan > > > > You're getting this email because you were associated with the identified > > lines of code (noted below) that were touched by commits: > > > > Mon Aug 31 19:10:08 2020 -0400 > > a0922278f83e ("drm/nouveau/kms/nv50-: Refactor and cleanup DP HPD handling") > > Hi Kees, this looks like a good idea, but maybe double check the Cc list > generation? I was Cc'd on four mails today that I thought were > irrelevant to me. Hi! Heh, I was recently asked to _expand_ the CC list. :) For these last pass of reports, I added a get_maintainers.pl run to the identified commit. In this instance, the commit touched: drivers/gpu/drm/nouveau/dispnv04/disp.c | 6 + drivers/gpu/drm/nouveau/dispnv50/disp.c | 192 ++++++++++++++++++++++-------------------------- drivers/gpu/drm/nouveau/nouveau_connector.c | 14 --- drivers/gpu/drm/nouveau/nouveau_display.c | 2 drivers/gpu/drm/nouveau/nouveau_display.h | 2 drivers/gpu/drm/nouveau/nouveau_dp.c | 132 ++++++++++++++++++++++++++++----- drivers/gpu/drm/nouveau/nouveau_encoder.h | 33 +++++++- 7 files changed, 244 insertions(+), 137 deletions(-) And the get_maintainers.pl rationale was: Ben Skeggs (supporter:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS,commit_signer:1/1=100%,commit_signer:6/16=38%,authored:4/16=25%,added_lines:23/124=19%,removed_lines:36/152=24%) Karol Herbst (supporter:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS,commit_signer:2/1=100%) Lyude Paul (supporter:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS,commit_signer:9/16=56%,authored:6/16=38%,added_lines:92/124=74%,removed_lines:107/152=70%) David Airlie (maintainer:DRM DRIVERS) Daniel Vetter (maintainer:DRM DRIVERS) Ilia Mirkin (commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,removed_lines:2/2=100%) "Nathan E. Egge" (commit_signer:1/1=100%) Jani Nikula (commit_signer:6/16=38%) Dave Airlie (commit_signer:5/16=31%) Thomas Zimmermann (commit_signer:4/16=25%,authored:4/16=25%) dri-devel@lists.freedesktop.org (open list:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS) nouveau@lists.freedesktop.org (open list:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS) -- Kees Cook