Re: [PATCH 1/3] initial parsing of __attribute__((format))

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27/10/18 00:13, Luc Van Oostenryck wrote:
On Fri, Oct 26, 2018 at 04:26:30PM +0100, Ben Dooks wrote:
---
  parse.c  | 75 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-
  symbol.h |  2 ++
  2 files changed, 76 insertions(+), 1 deletion(-)



Another reason why printf_va_start & printf_msg need to move (to ctx->ctype) is
that the attribute can be placed before the function. For example:
	extern __attribute__((format(printf, 1, 2))) void printk(const char *, ...);
is legit but gives here the error "no base context for format".

-- Luc

Ok, thanks for the review.
I will try and get a new series out this week.



--
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

https://www.codethink.co.uk/privacy.html



[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux