Received: by 10.223.176.5 with SMTP id f5csp2774745wra; Mon, 5 Feb 2018 09:37:24 -0800 (PST) X-Google-Smtp-Source: AH8x227ccSpWgPjQgoCLs+Nqy1mhRkuP9K0hr43aQujYwJ9uK/cKzagD2BsStKpOte4OplbrFf66 X-Received: by 10.99.122.93 with SMTP id j29mr8783630pgn.351.1517852244145; Mon, 05 Feb 2018 09:37:24 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1517852244; cv=none; d=google.com; s=arc-20160816; b=QZcFA0NdEll2bY011yxTOqMBpFhzB8iW8pzYfGSG0LeFRcoDpMLZc5q63XmLqGprRC aUH2naFop502Mlfv8GG43KmeAucx83JekoxNJv8bHhzpTwQNlj4W8+D9fFGuxsrmuRwt F83n51hJUMAeqmcikSfJuMllCOhGr1h5dMJgQrhItUHVr2FDqx/9OpnJYiMW07+JxUKN 5EcfFHN+E5Kh/V13tkPbSTIsL3ox0H2VC9KlG2m1QdRafoFoOvDXhvVLmobl288wKmIk ybbx5ATIbEh8ZN6psOHD3lqELMjL5hbLX1xBbe09Nbz9TeMay4IO+5tTYrWK2jPp28O0 Fsvg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature :arc-authentication-results; bh=H1o4PhJUm6XqUweWc5gMd9hR/ZkrB7oKcQcEQoMES/A=; b=aXYk4UzLokKcRjik0xji6U+NmRM3N2J2baVnXagjZm+1DwSN+akVIZb9rF0E+nRtp0 o4frpvZdSc4rT1Bey7zBnDmjyPTD2CfMe6WmaeH9+9oF0khBQvZa1bDl7Gno3QaAcFI8 hvWsaFlfIxXt49JONtrJ/qddCvH5e8CmFU+kMCW7MuHw2rbhzFnJ+p7C4NW5CTQFxr2a ieJqLgLdjpEWxqz8CGXS/xoOvKvY17VtKv1Bq7pzmLWAn87eWagtH87O7ycoMOSghq4a t3BJeTHsAl4YqhPLrsRNp9up+CsW3wsMbC676SSxHvqlwzrpDIWYjzhh6XHHN2RU0oD5 6How== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@infradead.org header.s=merlin.20170209 header.b=eYjE8Ohl; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y18si336211pfe.38.2018.02.05.09.37.09; Mon, 05 Feb 2018 09:37:24 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@infradead.org header.s=merlin.20170209 header.b=eYjE8Ohl; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753335AbeBERgb (ORCPT + 99 others); Mon, 5 Feb 2018 12:36:31 -0500 Received: from merlin.infradead.org ([205.233.59.134]:49074 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752999AbeBERgZ (ORCPT ); Mon, 5 Feb 2018 12:36:25 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=H1o4PhJUm6XqUweWc5gMd9hR/ZkrB7oKcQcEQoMES/A=; b=eYjE8OhlIC8LhuIul1vEV2K0X9 t51LylYtK6+6Y5t5ixOl9EtXSavBatkVVr37HjQMIKU2X3OXmYWjiAC1inXxGh/jo+W+YeTmBiST/ WVcgy0fstPITdiUjiRIk3w51kSS4CiwEyMH2bbsfmOZvzy8lVINllZc9gtoaXiW4cuBDAJ6FKqAyr 1Dp5NOFiVOZxpO0UsJpf5CXkhtnAIKYZnVJ7Au1WP+ojp65Eb0LS6Dn+qjtNUuIZT1++Sgs5Raw3a AUyVaPLlqZpfvixhgl7byH/8eSNeLypN/AeycOa0zb2W09mvHnLOh8GEj3NDWnkAQdO23MdQE6i9+ qJDW/AOQ==; Received: from static-50-53-52-16.bvtn.or.frontiernet.net ([50.53.52.16] helo=dragon.site) by merlin.infradead.org with esmtpsa (Exim 4.89 #1 (Red Hat Linux)) id 1eikgZ-0000u5-7P; Mon, 05 Feb 2018 17:36:07 +0000 Subject: Re: [PATCH] vsprintf: avoid misleading "(null)" for %px To: Steven Rostedt , Adam Borowski Cc: "Tobin C. Harding" , Petr Mladek , Sergey Senozhatsky , linux-kernel@vger.kernel.org, Andrew Morton , Joe Perches , Kees Cook , "Roberts, William C" , Linus Torvalds , David Laight , Geert Uytterhoeven References: <20180204174521.21383-1-kilobyte@angband.pl> <20180205094438.pfd7ffymlvklpxe7@pathway.suse.cz> <20180205100305.GO29988@eros> <20180205152218.hxgozi67zka4hgkf@angband.pl> <20180205114952.6af99dff@gandalf.local.home> From: Randy Dunlap Message-ID: <49fbcc8b-9522-f19c-d51c-d0059445abdc@infradead.org> Date: Mon, 5 Feb 2018 09:36:03 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <20180205114952.6af99dff@gandalf.local.home> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/05/2018 08:49 AM, Steven Rostedt wrote: > On Mon, 5 Feb 2018 16:22:19 +0100 > Adam Borowski wrote: > >> My change touches %px only, where your concern doesn't apply. >> >> You're right, though, when it comes to %pK: >> printk("%%pK: %pK, %%px: %px\n", 0, 0); >> says >> %pK: 00000000ba8bdc0a, %px: 0000000000000000 >> >> So what should we do? Avoid hashing 0? Print a special value? > > My personal opinion is that NULL should stay NULL and not be hashed. > What security issue could be leaked by a NULL? I'm not a security > person, that's a real question. Agree. -- ~Randy