[U-Boot] [PATCH] imx: mx6/7 define CONFIG_SYS_VSNPRINTF

Peng Fan
2016-01-14T01:54:09Z

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