diff options
author | Steven Rostedt <rostedt@goodmis.org> | 2025-07-18 16:18:44 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2025-07-21 16:31:04 -0400 |
commit | 61f7e318e99d3b398670518dd3f4f8510d1800fc (patch) | |
tree | 9892602f9cb639ca78bc4c1157eab72574a98425 /scripts/lib/kdoc/kdoc_parser.py | |
parent | acd98e230ee86a9812c56a081929248cdbe412fa (diff) |
ktest.pl: Prevent recursion of default variable options
If a default variable contains itself, do not recurse on it.
For example:
ADD_CONFIG := ${CONFIG_DIR}/temp_config
DEFAULTS
ADD_CONFIG = ${CONFIG_DIR}/default_config ${ADD_CONFIG}
The above works because the temp variable ADD_CONFIG (is a temp because it
is created with ":=") is already defined, it will be substituted in the
variable option. But if it gets commented out:
# ADD_CONFIG := ${CONFIG_DIR}/temp_config
DEFAULTS
ADD_CONFIG = ${CONFIG_DIR}/default_config ${ADD_CONFIG}
Then the above will go into a recursive loop where ${ADD_CONFIG} will
get replaced with the current definition of ADD_CONFIG which contains the
${ADD_CONFIG} and that will also try to get converted. ktest.pl will error
after 100 attempts of recursion and fail.
When replacing a variable with the default variable, if the default
variable contains itself, do not replace it.
Cc: "John Warthog9 Hawley" <warthog9@kernel.org>
Cc: Dhaval Giani <dhaval.giani@gmail.com>
Cc: Greg KH <gregkh@linuxfoundation.org>
Link: https://lore.kernel.org/20250718202053.732189428@kernel.org
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'scripts/lib/kdoc/kdoc_parser.py')
0 files changed, 0 insertions, 0 deletions