Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/xlint/xlint usr.bin/xlint: Fix -Werror=format-overfl...



details:   https://anonhg.NetBSD.org/src/rev/a5ca50f8680c
branches:  trunk
changeset: 969142:a5ca50f8680c
user:      fox <fox%NetBSD.org@localhost>
date:      Sun Feb 09 08:10:25 2020 +0000

description:
usr.bin/xlint: Fix -Werror=format-overflow= error.

Replace sprintf(3) with snprintf(3).

Error was reported when build.sh was run with MKLIBCSANITIZER=yes flag.

Reviewed by: kamil@

diffstat:

 usr.bin/xlint/xlint/xlint.c |  12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diffs (41 lines):

diff -r 53e984f1cbac -r a5ca50f8680c usr.bin/xlint/xlint/xlint.c
--- a/usr.bin/xlint/xlint/xlint.c       Sun Feb 09 08:09:58 2020 +0000
+++ b/usr.bin/xlint/xlint/xlint.c       Sun Feb 09 08:10:25 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: xlint.c,v 1.47 2019/04/13 15:08:49 christos Exp $ */
+/* $NetBSD: xlint.c,v 1.48 2020/02/09 08:10:25 fox Exp $ */
 
 /*
  * Copyright (c) 1996 Christopher G. Demetriou.  All Rights Reserved.
@@ -38,7 +38,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: xlint.c,v 1.47 2019/04/13 15:08:49 christos Exp $");
+__RCSID("$NetBSD: xlint.c,v 1.48 2020/02/09 08:10:25 fox Exp $");
 #endif
 
 #include <sys/param.h>
@@ -613,7 +613,7 @@
        const   char *bn, *suff;
        char    **args, *ofn, *pathname;
        const char *CC;
-       size_t  len;
+       size_t  buff_size, len;
        int is_stdin;
        int     fd;
 
@@ -648,10 +648,10 @@
                        warnx("-i not supported without -o for standard input");
                        return;
                }
-               ofn = xmalloc(strlen(bn) + (bn == suff ? 4 : 2));
+               buff_size = strlen(bn) + (bn == suff ? 4 : 2);
+               ofn = xmalloc(buff_size);
                len = bn == suff ? strlen(bn) : (size_t)((suff - 1) - bn);
-               (void)sprintf(ofn, "%.*s", (int)len, bn);
-               (void)strcat(ofn, ".ln");
+               (void)snprintf(ofn, buff_size, "%.*s.ln", (int)len, bn);
        } else {
                ofn = xmalloc(strlen(tmpdir) + sizeof ("lint1.XXXXXX"));
                (void)sprintf(ofn, "%slint1.XXXXXX", tmpdir);



Home | Main Index | Thread Index | Old Index