Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Wed, 11 Dec 2002 12:07:08 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Wed, 11 Dec 2002 12:07:08 -0500 Received: from x35.xmailserver.org ([208.129.208.51]:38534 "EHLO x35.xmailserver.org") by vger.kernel.org with ESMTP id ; Wed, 11 Dec 2002 12:07:06 -0500 X-AuthUser: davidel@xmailserver.org Date: Wed, 11 Dec 2002 09:16:54 -0800 (PST) From: Davide Libenzi X-X-Sender: davide@blue1.dev.mcafeelabs.com To: Robert Love cc: Linux Kernel Mailing List Subject: Re: [PATCH] epoll: don't printk pointer value In-Reply-To: <1039588045.833.3.camel@phantasy> Message-ID: References: <1039588045.833.3.camel@phantasy> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 868 Lines: 25 On Tue, 11 Dec 2002, Robert Love wrote: > Davide, > > I really cannot think of a good reason why eventpoll_init() should print > a pointer value to user-space - especially the value of current? > > I do not think this is good practice and someone might even consider it > a security hole. Personally, I would prefer to remove the "successfully > initialized" message altogether, but at the very least can we not print > current's address? It's ok Robert, it was used for debugging purposes and now it can be completely removed ( the whole printk() ). I'll post a patch to Linus that removes the printk() ... - Davide - 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/