Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-21 Thread Kyrill Tkachov
On 20/09/16 18:56, Pedro Alves wrote: On 09/20/2016 05:49 PM, Kyrill Tkachov wrote: Ok, I'm proposing a new function defined in system.h (though I'm open to suggestions for other location). It would be something like: static inline int ATTRIBUTE_PRINTF_3 gcc_snprintf (char *str, size_t size,

Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-20 Thread Pedro Alves
On 09/20/2016 05:49 PM, Kyrill Tkachov wrote: > Ok, I'm proposing a new function defined in system.h > (though I'm open to suggestions for other location). > It would be something like: > > static inline int ATTRIBUTE_PRINTF_3 > gcc_snprintf (char *str, size_t size, const char *format, ...) > {

Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-20 Thread Richard Earnshaw (lists)
On 20/09/16 17:49, Kyrill Tkachov wrote: > > On 20/09/16 15:13, Richard Earnshaw (lists) wrote: >> On 08/09/16 12:00, Kyrill Tkachov wrote: >>> Hi all, >>> >>> This is a rebase and slight respin of [1] that I sent out last November >>> to change all uses of >>> sprintf to snprintf in the arm

Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-20 Thread Kyrill Tkachov
On 20/09/16 15:13, Richard Earnshaw (lists) wrote: On 08/09/16 12:00, Kyrill Tkachov wrote: Hi all, This is a rebase and slight respin of [1] that I sent out last November to change all uses of sprintf to snprintf in the arm backend. Bootstrapped and tested on arm-none-linux-gnueabihf. Ok

Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-20 Thread Richard Earnshaw (lists)
On 08/09/16 12:00, Kyrill Tkachov wrote: > Hi all, > > This is a rebase and slight respin of [1] that I sent out last November > to change all uses of > sprintf to snprintf in the arm backend. > > Bootstrapped and tested on arm-none-linux-gnueabihf. > Ok for trunk? > > Thanks, > Kyrill > > [1]

Re: [PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-20 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00429.html Thanks, Kyrill On 08/09/16 12:00, Kyrill Tkachov wrote: Hi all, This is a rebase and slight respin of [1] that I sent out last November to change all uses of sprintf to snprintf in the arm backend. Bootstrapped and tested on

[PATCH][ARM][v2] Use snprintf rather than sprintf

2016-09-08 Thread Kyrill Tkachov
Hi all, This is a rebase and slight respin of [1] that I sent out last November to change all uses of sprintf to snprintf in the arm backend. Bootstrapped and tested on arm-none-linux-gnueabihf. Ok for trunk? Thanks, Kyrill [1] https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00937.html