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....@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....@nxp.com> >> Cc: Peter Robinson <pbrobin...@gmail.com> >> Cc: Fabio Estevam <fabio.este...@freescale.com> >> Cc: Adrian Alonso <aalo...@freescale.com> >> Cc: Stefano Babic <sba...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
