Skip to content

Commit 93b09fd

Browse files
authored
Merge pull request #1466 from j-t-1/configuration
Tweak configuration.py
2 parents 1f91fd4 + 79fe1c5 commit 93b09fd

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

volatility3/framework/interfaces/configuration.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
def path_join(*args) -> str:
5454
"""Joins configuration paths together."""
5555
# If a path element (particularly the first) is empty, then remove it from the list
56-
args = tuple([arg for arg in args if arg])
56+
args = tuple(arg for arg in args if arg)
5757
return CONFIG_SEPARATOR.join(args)
5858

5959

@@ -772,8 +772,7 @@ def make_subconfig(
772772
str: The newly generated full configuration path
773773
"""
774774
random_config_dict = "".join(
775-
random.SystemRandom().choice(string.ascii_uppercase + string.digits)
776-
for _ in range(8)
775+
random.SystemRandom().choices(string.ascii_uppercase + string.digits, k=8)
777776
)
778777
new_config_path = path_join(base_config_path, random_config_dict)
779778
# TODO: Check that the new_config_path is empty, although it's not critical if it's not since the values are merged in

0 commit comments

Comments
 (0)