Source-Changes-D archive

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

Re: src/sys/modules/spdmem



On Thu, Aug 18, 2011 at 11:11:20AM -0700, Paul Goyette wrote:
> >Module Name:    src
> >Committed By:   christos
> >Date:           Thu Aug 18 17:02:49 UTC 2011
> >
> >Modified Files:
> >        src/sys/modules/spdmem: Makefile
> >
> >Log Message:
> >document non-literal string format
> 
> Rather than sweeping the issue under the rug, wouldn't it be better to
> actually fix the problem?
> 
> See attached diff which replaces the "variable" format with a
> literal #define string ...

I think we should make no changes to appease the compiler in this case.
There is nothing inherently safer about using a literal format string
than a static const format string, the compiler just isn't smart enough
to tell an unsafe non-literal format string from a safe one.

Dave

-- 
David Young             OJC Technologies
dyoung%ojctech.com@localhost      Urbana, IL * (217) 344-0444 x24


Home | Main Index | Thread Index | Old Index