Hi Tom,
On Wed, Jan 13, 2016 at 02:57:00PM -0500, Tom Rini wrote:
On Tue, Jan 12, 2016 at 05:45:24PM +0800, Peng Fan wrote:
From: Peng Fan peng.fan@nxp.com
Define CONFIG_SYS_VSNPRINTF to use snprintf. If not define CONFIG_SYS_VSNPRINTF, snprintf is sprintf.
Report by Coverity: pass string init_val of unknown size to sprintf
Signed-off-by: Peng Fan peng.fan@nxp.com Cc: Peter Robinson pbrobinson@gmail.com Cc: Fabio Estevam fabio.estevam@freescale.com Cc: Adrian Alonso aalonso@freescale.com Cc: Stefano Babic sbabic@denx.de
I'm not sure how I feel about fixing this range of issues only for certain targets. I need to do some global building here and see what happens.
With CONFIG_SYS_VSNPRINTF defined, image size may be bigger. So here, I use #ifdef CONFIG_SPL_BUILD.
Ok, wait for your further comments on this.
Regards, Peng.
-- Tom