This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3] getrandom system call wrapper [BZ #17252]


On 09/08/2016 11:32 AM, Florian Weimer wrote:
>> Using __asm__ to change the assembler name of the C-level symbol is probably
>> preferred for __GCC__.
> 
> This does not work in all cases because a getrandom function definition will
> inherit the asm alias on the prototype and interpose the supposedly-protected
> symbol.
> 
> But now that we have a <sys/random.h> header, maybe this isn't a problem—if an
> application includes the <sys/random.h> header, it shows that it is aware of
> getrandom, so it's less likely that it's interposing the function accidentally.

Indeed, that's what I was assuming.  I hadn't read the previous patches where
this header didn't exist.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]