Source-Changes-HG archive

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

[src/trunk]: src/usr.bin/make/unit-tests tests/make: document use-after-free ...



details:   https://anonhg.NetBSD.org/src/rev/9a2d04d9eb9c
branches:  trunk
changeset: 1026755:9a2d04d9eb9c
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Dec 05 15:51:33 2021 +0000

description:
tests/make: document use-after-free in ':@' more precisely

The commit message from the previous commit was misleading.  This bug
had not been introduced in 2016.02.27.16.20.06 (which was entirely
unrelated) but rather in 2016.02.18.23.33.25.

The commit 2016.02.27.16.20.06 just happened to be the first one from my
make-archive that triggered the bug.  That's because I had only checked
out the last commit from each month, to save some disk space.

diffstat:

 usr.bin/make/unit-tests/varmod-loop-delete.mk |  8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diffs (21 lines):

diff -r 1a3f96ae05f6 -r 9a2d04d9eb9c usr.bin/make/unit-tests/varmod-loop-delete.mk
--- a/usr.bin/make/unit-tests/varmod-loop-delete.mk     Sun Dec 05 15:25:56 2021 +0000
+++ b/usr.bin/make/unit-tests/varmod-loop-delete.mk     Sun Dec 05 15:51:33 2021 +0000
@@ -1,13 +1,13 @@
-# $NetBSD: varmod-loop-delete.mk,v 1.1 2021/12/05 15:20:13 rillig Exp $
+# $NetBSD: varmod-loop-delete.mk,v 1.2 2021/12/05 15:51:33 rillig Exp $
 #
 # Tests for the variable modifier ':@', which as a side effect allows to
 # delete an arbitrary variable.
 
 # A side effect of the modifier ':@' is that the loop variable is created as
 # an actual variable in the current evaluation scope (Command/Global/target),
-# and at the end of the loop, this variable is deleted.  Before var.c 1.963
-# from 2021-12-05, a variable could be deleted while it was in use, leading to
-# a use-after-free bug.
+# and at the end of the loop, this variable is deleted.  Since var.c 1.204
+# from 2016-02-18 and before var.c 1.963 from 2021-12-05, a variable could be
+# deleted while it was in use, leading to a use-after-free bug.
 #
 # See Var_Parse, comment 'the value of the variable must not change'.
 



Home | Main Index | Thread Index | Old Index