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 CA98BEB64DD for ; Thu, 13 Jul 2023 06:01:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232068AbjGMGBW (ORCPT ); Thu, 13 Jul 2023 02:01:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37194 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232430AbjGMGBT (ORCPT ); Thu, 13 Jul 2023 02:01:19 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7CCA12101 for ; Wed, 12 Jul 2023 23:01:14 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 0D37B61A1B for ; Thu, 13 Jul 2023 06:01:14 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DBE8AC433C7; Thu, 13 Jul 2023 06:01:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1689228073; bh=w+P5mRmNhuaIFNb8g78p5+MFYDSB8dr91PCPz9ZsAXE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JL/vxpsWci8zWey466CpqtRF1IqCZhLnR6XndgpkytQkfb/silNbeElL0xnXg92kj 1VJH8awQwE24zQ3y0UPNbRuOCFnpe465ubXwPf+ped/wR1yALdAnGAFsvdWKmTWcLU V2p421BzdUFiZGHQDiSxnfnPijJM2FTjM0vfhEdI= Date: Thu, 13 Jul 2023 08:01:08 +0200 From: Greg Kroah-Hartman To: =?utf-8?B?546L5piOLei9r+S7tuW6leWxguaKgOacr+mDqA==?= Cc: David Hildenbrand , Akinobu Mita , Andrew Morton , LKML , "opensource.kernel" Subject: Re: =?utf-8?B?5Zue5aSNOiBbUEFUQ0ggdjFdIGxp?= =?utf-8?Q?b=3AFix_an_NUL?= =?utf-8?Q?L?= vs IS_ERR() bug for debugfs_create_dir() in err_inject_init() Message-ID: <2023071334-tattle-mobile-dab4@gregkh> References: <20230712135226.10041-1-machel@vivo.com> <2023071202-varsity-evasion-580b@gregkh> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Thu, Jul 13, 2023 at 02:09:03AM +0000, 王明-软件底层技术部 wrote: > Hi greg k-h > I proposed this patch because I found that debugfs_create_dir() does > not return NULL, but returns an error pointer, executing if statements > according to the original code meaning no error, so I made this change > : ) That's fine, but you should fix this up to use the api properly if you are making these types of changes, otherwise we just have to do it again in the future. thanks, greg k-h