[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Replace vsprintf with vsnprintf in ext2fs/msg.c
From: |
Samuel Thibault |
Subject: |
Re: [PATCH] Replace vsprintf with vsnprintf in ext2fs/msg.c |
Date: |
Wed, 9 Nov 2022 15:06:46 +0100 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
Applied, thanks!
Flavio Cruz, le mer. 09 nov. 2022 00:49:04 -0500, a ecrit:
> ---
>
> Hi
>
> On Sun, Nov 06, 2022 at 11:45:19AM +0100, Samuel Thibault wrote:
> > Hello,
> >
> > Flavio Cruz, le dim. 06 nov. 2022 01:23:00 -0400, a ecrit:
> > > ---
> > > ext2fs/msg.c | 9 +++++----
> > > 1 file changed, 5 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/ext2fs/msg.c b/ext2fs/msg.c
> > > index 83939b06..af636b7c 100644
> > > --- a/ext2fs/msg.c
> > > +++ b/ext2fs/msg.c
> > > @@ -38,7 +38,8 @@ int printf (const char *fmt, ...)
> > > return done;
> > > }
> > >
> > > -static char error_buf[1024];
> > > +#define ERROR_BUF_SIZE 1024
> > > +static char error_buf[ERROR_BUF_SIZE];
> > >
> > > void _ext2_error (const char * function, const char * fmt, ...)
> > > {
> > > @@ -47,7 +48,7 @@ void _ext2_error (const char * function, const char *
> > > fmt, ...)
> > > pthread_mutex_lock (&printf_lock);
> > >
> > > va_start (args, fmt);
> > > - vsprintf (error_buf, fmt, args);
> > > + vsnprintf (error_buf, ERROR_BUF_SIZE, fmt, args);
> >
> > Please rather use sizeof (error_buf).
>
> Done
>
> ext2fs/msg.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/ext2fs/msg.c b/ext2fs/msg.c
> index 83939b06..b6b1a155 100644
> --- a/ext2fs/msg.c
> +++ b/ext2fs/msg.c
> @@ -47,7 +47,7 @@ void _ext2_error (const char * function, const char * fmt,
> ...)
> pthread_mutex_lock (&printf_lock);
>
> va_start (args, fmt);
> - vsprintf (error_buf, fmt, args);
> + vsnprintf (error_buf, sizeof (error_buf), fmt, args);
> va_end (args);
>
> fprintf (stderr, "ext2fs: %s: %s: %s\n", diskfs_disk_name, function,
> error_buf);
> @@ -62,7 +62,7 @@ void _ext2_panic (const char * function, const char * fmt,
> ...)
> pthread_mutex_lock (&printf_lock);
>
> va_start (args, fmt);
> - vsprintf (error_buf, fmt, args);
> + vsnprintf (error_buf, sizeof (error_buf), fmt, args);
> va_end (args);
>
> fprintf(stderr, "ext2fs: %s: panic: %s: %s\n",
> @@ -80,7 +80,7 @@ void ext2_warning (const char * fmt, ...)
> pthread_mutex_lock (&printf_lock);
>
> va_start (args, fmt);
> - vsprintf (error_buf, fmt, args);
> + vsnprintf (error_buf, sizeof (error_buf), fmt, args);
> va_end (args);
>
> fprintf (stderr, "ext2fs: %s: warning: %s\n", diskfs_disk_name, error_buf);
> --
> 2.37.2
>
--
Samuel
---
Pour une évaluation indépendante, transparente et rigoureuse !
Je soutiens la Commission d'Évaluation de l'Inria.