Received: by 10.223.185.116 with SMTP id b49csp758506wrg; Wed, 14 Feb 2018 06:36:43 -0800 (PST) X-Google-Smtp-Source: AH8x226KylfHedTHCy1HLhU8dEf9Z+AoUl+pRkbKmykq0BZo2Lcx6yJan/szD6t7+scrRNJG62zn X-Received: by 10.99.113.7 with SMTP id m7mr4050038pgc.403.1518619003800; Wed, 14 Feb 2018 06:36:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1518619003; cv=none; d=google.com; s=arc-20160816; b=d79nAfwWqguud+nCnfn5TO++3Amcc1M87BON6xamDIAj9KxPiGHfZDvCq1ByJ3OKqq vwWroIeWHaXV7lvAggV4HkQeMyiWzT7PdyxUBi7YMyOjPSy2U14NdNDNtq4/9p95bRP6 8gpLpIEwTz4vb6R9Ka8LOmjkqKgDvF6YGo7WQrqWXOkGtQExeKoDq1KYl5xuPQoElCDl Ze3MmrHe0IyNRKGRjDiMsIfJD0olEl69hk1cx50rhWjh0G+Bz4aCwtuNGs02SsrQsDc1 OEc+FihVeMsL5Ycd//OK4SegJyqxoGSWt/bJwFymOqUIQwNciNkEtv49MVJ36q2vzZCa a2CQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=N5qFSjor6AycpYIYVsn7//q5Ls1KW10NmIyK8C+vaxM=; b=T3qQhd2c/ihvtZaZ8xGmNUoTShHlYJ9JVWAcsl81iGDeRJ40UYgfTchthdaQZndzVu +aBZpH6HWadCKoC6WtyIbjtZOgMf5/WNmE+DDB4boElSQeg3FqD+AsigUOMAfD9nErCu sU3qY2N27xV8BwvYXP2IZgAM42XZqOlqrVNBJrzh5AYZGFXw0BmKMRfKERo/UNrVXPTe sJLefby/o2FmmtWuHF1IV8LYUD5h9tJic8HnL8AwDNsVAg8UxzFyaJl1Q4b831Ys7iIQ SvrV94rYw6VL33ceewIHIF+b/SU6b5fFImvI310fTuTtel3jg+pQ5+6EjWkXekmGEUOg +nwg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=gvtoy7rB; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id z11si9738584pfj.351.2018.02.14.06.36.25; Wed, 14 Feb 2018 06:36:43 -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=pass header.i=@gmail.com header.s=20161025 header.b=gvtoy7rB; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030872AbeBNOfh (ORCPT + 99 others); Wed, 14 Feb 2018 09:35:37 -0500 Received: from mail-qt0-f178.google.com ([209.85.216.178]:40193 "EHLO mail-qt0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030381AbeBNOfg (ORCPT ); Wed, 14 Feb 2018 09:35:36 -0500 Received: by mail-qt0-f178.google.com with SMTP id c19so7907486qtm.7 for ; Wed, 14 Feb 2018 06:35:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=N5qFSjor6AycpYIYVsn7//q5Ls1KW10NmIyK8C+vaxM=; b=gvtoy7rBBsK6YfmMEWGXpsLkl3+3xqCnIEUqrq+XCeKzm/Ude09Oggrq77NCXR2Mg5 t3Q84qvswuSVL07OhGJCeZHV8dQyeHu9iBYANt7X9MTDZUJYQdAEQ88khaWni2HdIQ02 embqp3RkdhXzWU4gmss9EeBvimzVLPLj5bKAqdi01xk2RRN88Abh+AHwBEV98DhFJvWJ A6c7QMYXHxpT/GfEdfmhNnUaDPo8EkmsrqyroGt59AOy2GUlTxfygzymmS1kiyMyCcnH zCqFs5jRHgQ3xtnYz9qlofvGNSuiizoix4g8434NDcdE/4likEfWGUtXkxqgw0RtZRwB BLeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=N5qFSjor6AycpYIYVsn7//q5Ls1KW10NmIyK8C+vaxM=; b=sIDWg0o0O5ssTi3A35uUcMfsNSOpv6L9pE/6G0h/JUuGVfZZGOb5/BNh6Vr6Aelf3W kmAe57gSHsBtzuSdkaahrU24ChpZqk1eAMAuqc+npshSAMdn+Z1eEWh1RLNHIGl7Cgr6 wxMne5r0BNLjRK4mrE8iII0C1dF8+eMPCWt5dl7K4srEVlnK0u9T9x2OusfQIIogpZ5f ZLv/9icTshB5HUa+9dUmDmrAT7RSnaxNyH+ky+wctHF2TxStRmZuxDVRROqz0jwv/VN2 nhvXTu9Mm9LR9EAi9UVTrHk+OJFnqWH3jD8E/YoNXFXJHEXXf42jkRPi7Klw1av198GK OyYQ== X-Gm-Message-State: APf1xPBayk5O7hG17gUsgXr/gkgV6BlAXKg+kbQ2MvNF0seZxJqHSvYO 8L72LYINW/YeF1GXmHEJMNSvl9cOcLvWkEioB/w= X-Received: by 10.200.11.13 with SMTP id e13mr2906806qti.340.1518618935793; Wed, 14 Feb 2018 06:35:35 -0800 (PST) MIME-Version: 1.0 Received: by 10.12.195.82 with HTTP; Wed, 14 Feb 2018 06:35:35 -0800 (PST) In-Reply-To: <20180209120313.rckq3s54nr6yyus4@pathway.suse.cz> References: <20180204174521.21383-1-kilobyte@angband.pl> <20180205094438.pfd7ffymlvklpxe7@pathway.suse.cz> <20180205201555.GQ29988@eros> <20180205205817.72dy7e7xzjcnwmhs@angband.pl> <20180207150328.nuk7f76nutq2trcg@pathway.suse.cz> <20180207154116.xvojw3bzmnmexkav@pathway.suse.cz> <20180209120313.rckq3s54nr6yyus4@pathway.suse.cz> From: Andy Shevchenko Date: Wed, 14 Feb 2018 16:35:35 +0200 Message-ID: Subject: Re: [PATCH] vsprintf: avoid misleading "(null)" for %px To: Petr Mladek Cc: Geert Uytterhoeven , Adam Borowski , Kees Cook , "Tobin C. Harding" , Sergey Senozhatsky , Steven Rostedt , LKML , Andrew Morton , Joe Perches , "Roberts, William C" , Linus Torvalds , David Laight , Randy Dunlap Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 9, 2018 at 2:03 PM, Petr Mladek wrote: > On Thu 2018-02-08 17:29:14, Andy Shevchenko wrote: >> On Wed, Feb 7, 2018 at 5:41 PM, Petr Mladek wrote: >> > On Wed 2018-02-07 16:11:13, Geert Uytterhoeven wrote: >> >> > To make it clear. I was talking about "%p" format that is handled >> > in the pointer() function in lib/vsprintf.c. The "(null)" makes >> > sense only for the many modifiers that do deference of >> > the pointer, e.g. "%pa", "%pE", "%ph". >> >> JFYI: I have a patch to eliminate those for %pE & %ph. >> >> Google for "lib/vsprintf: Remove useless NULL checks" as a first in >> the series for new extension to print times. > > I am slightly confused. IMHO, it makes sense to printk "(null)" > for %pE and %ph. Yes, but isn't it done by if (!ptr) check in the very beginning of the pointer() helper? > Or do you just want to avoid the duplicit check in hex_string() > and escaped_string()? And that is as well. > Well, it might be better to discuss this once you send the patch. I can Cc you, though the patch is pretty independent on the series. I can send it separately. -- With Best Regards, Andy Shevchenko