On Fri, 2019-08-30 at 23:46 +0200, Rasmus Villemoes wrote: > It has been suggested several times to extend vsnprintf() to be able > to convert the numeric value of ENOSPC to print "ENOSPC". This is yet > another attempt. Rather than adding another %p extension, simply teach > plain %p to convert ERR_PTRs. While the primary use case is
Thanks, this all this seems reasonable except for > diff --git a/lib/vsprintf.c b/lib/vsprintf.c [] > @@ -2178,8 +2204,6 @@ char *pointer(const char *fmt, char *buf, char *end, > void *ptr, > return flags_string(buf, end, ptr, spec, fmt); > case 'O': > return kobject_string(buf, end, ptr, spec, fmt); > - case 'x': > - return pointer_string(buf, end, ptr, spec); > } > > /* default is to _not_ leak addresses, hash before printing */ why remove this?