Source-Changes-HG archive

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

[src/trunk]: src/usr.bin/indent indent: mark obviously broken code



details:   https://anonhg.NetBSD.org/src/rev/2c83727052a9
branches:  trunk
changeset: 1024595:2c83727052a9
user:      rillig <rillig%NetBSD.org@localhost>
date:      Fri Oct 29 18:50:52 2021 +0000

description:
indent: mark obviously broken code

diffstat:

 usr.bin/indent/indent.c |  10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diffs (31 lines):

diff -r 15f7834450d0 -r 2c83727052a9 usr.bin/indent/indent.c
--- a/usr.bin/indent/indent.c   Fri Oct 29 18:18:03 2021 +0000
+++ b/usr.bin/indent/indent.c   Fri Oct 29 18:50:52 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: indent.c,v 1.171 2021/10/29 17:50:37 rillig Exp $      */
+/*     $NetBSD: indent.c,v 1.172 2021/10/29 18:50:52 rillig Exp $      */
 
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: indent.c,v 1.171 2021/10/29 17:50:37 rillig Exp $");
+__RCSID("$NetBSD: indent.c,v 1.172 2021/10/29 18:50:52 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/indent.c 340138 2018-11-04 19:24:49Z oshogbo $");
 #endif
@@ -188,6 +188,12 @@
         * process_comment() will use that to calculate original indentation
         * of a boxed comment.
         */
+       /*
+        * FIXME: This '4' needs an explanation. For example, in the snippet
+        * 'if(expr)/''*comment', the 'r)' of the code is not copied. If there
+        * is an additional line break before the ')', memcpy tries to copy
+        * (size_t)-1 bytes.
+        */
        memcpy(sc_buf, inp.buf, (size_t)(inp.s - inp.buf) - 4);
        save_com = sc_buf + (inp.s - inp.buf - 4);
        save_com[0] = save_com[1] = ' ';



Home | Main Index | Thread Index | Old Index