Thread (33 messages) flat view 33 messages, 8 authors, 2019-02-14

Re: [PATCH] KEYS: trusted: fix -Wvarags warning

From: James Bottomley <hidden>
Date: 2018-10-12 15:46:34
Also in: keyrings, linux-integrity, lkml

On Fri, 2018-10-12 at 10:44 -0500, Denis Kenzior wrote:
Hi James,
quoted
quoted
So instead of having unsigned char h3, can't we simply have bool
h3 or unsigned int h3?
Given the ambiguity in the standards, the safe thing that will work
for all time and all potential compilers is a char *
All right.  You state this with certainty, but I'd still like you to 
educate me why?

 From the links provided in the patch it seems that one cannot pass 
char/float/short to va_start().  Fair enough.  So if we make h3 an 
unsigned int, the issue goes away, no?
For the current version of clang, yes.  However, if we're fixing this
for good a char * pointer is the only guaranteed thing because it
mirrors current use in printf.

James
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help