Source-Changes-D archive

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

Re: CVS commit: src/lib/libcrypt



On Tue, Nov 29, 2011 at 09:23:20PM +0100, Joerg Sonnenberger wrote:
> On Tue, Nov 29, 2011 at 09:09:38PM +0100, Matthias Drochner wrote:
> > Anyway, here is a first cut on an "explicit_bzero" function
> > which doesn't get optimized away. I've put it to src/common
> > because the same thing makes sense in the kernel too.
> > Comments?
> 
> Just make it an alias on memset. That's good enough. I think calling it
> something like "secure_memset" or "safe_memset" is better, but opinions
> might differ.

I'm not sure that 'secure' or 'safe' are good prefixes for any function name.
You need to say WHY is is better.

This is also true of the plethora of names for 'better' strcpy()
and sprintf() functions available on some platforms.

        David

-- 
David Laight: david%l8s.co.uk@localhost


Home | Main Index | Thread Index | Old Index