Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Sat, 1 Dec 2001 07:37:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Sat, 1 Dec 2001 07:37:05 -0500 Received: from lightning.swansea.linux.org.uk ([194.168.151.1]:55305 "EHLO the-village.bc.nu") by vger.kernel.org with ESMTP id ; Sat, 1 Dec 2001 07:36:54 -0500 Subject: Re: [PATCH] if (foo) kfree(foo) /fs cleanup To: zwane@linux.realnet.co.sz (Zwane Mwaikambo) Date: Sat, 1 Dec 2001 12:45:26 +0000 (GMT) Cc: linux-kernel@vger.kernel.org (Linux Kernel) In-Reply-To: from "Zwane Mwaikambo" at Dec 01, 2001 01:36:54 PM X-Mailer: ELM [version 2.5 PL6] MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-Id: From: Alan Cox Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org IMHO this is precisely the wrong thing to do. We should make the = NULL check within kfree do a BUG() call. That way we fix the cases not being considered instead of hiding real bugs - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/