diff options
author | Mark Rutland <[email protected]> | 2022-04-27 18:31:20 +0100 |
---|---|---|
committer | Kees Cook <[email protected]> | 2022-05-08 01:33:08 -0700 |
commit | 1723d39d2fe49b8a75939060c95e7e908b463793 (patch) | |
tree | 952b398c8b3bac669c17fe46228d5e589098d682 /tools/perf/scripts/python/check-perf-trace.py | |
parent | 9ec79840d6afaf472294588a6bbe145bcdffa28b (diff) |
stackleak: clarify variable names
The logic within __stackleak_erase() can be a little hard to follow, as
`boundary` switches from being the low bound to the high bound mid way
through the function, and `kstack_ptr` is used to represent the start of
the region to erase while `boundary` represents the end of the region to
erase.
Make this a little clearer by consistently using clearer variable names.
The `boundary` variable is removed, the bounds of the region to erase
are described by `erase_low` and `erase_high`, and bounds of the task
stack are described by `task_stack_low` and `task_stack_high`.
As the same time, remove the comment above the variables, since it is
unclear whether it's intended as rationale, a complaint, or a TODO, and
is more confusing than helpful.
There should be no functional change as a result of this patch.
Signed-off-by: Mark Rutland <[email protected]>
Cc: Alexander Popov <[email protected]>
Cc: Andrew Morton <[email protected]>
Cc: Andy Lutomirski <[email protected]>
Cc: Kees Cook <[email protected]>
Signed-off-by: Kees Cook <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'tools/perf/scripts/python/check-perf-trace.py')
0 files changed, 0 insertions, 0 deletions