From c53966ae3f2f7f648177ed3bd0b56f16b7e79eb3 Mon Sep 17 00:00:00 2001 From: Philip Guyton Date: Tue, 6 Jun 2023 12:20:41 +0100 Subject: [PATCH] untested bytecast in smart.py #2564 Custom options autodev related byte assertion. --- src/rockstor/system/smart.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/rockstor/system/smart.py b/src/rockstor/system/smart.py index 3f685a032..600e39f68 100644 --- a/src/rockstor/system/smart.py +++ b/src/rockstor/system/smart.py @@ -446,8 +446,9 @@ def get_dev_options(dev_byid, custom_options=""): # target dev is not found, we can simply remove this option. # N.B. here we assume there is either 'autodev' or a specified target: # input validation was tested to reject both being entered. + # TODO: Needs validation post Preliminary python 3.6 port if "autodev" in dev_options: - dev_options.remove("autodev") + dev_options.remove(b"autodev") # If our custom options don't contain a raid controller target then add # the full path to our base device as our last device specific option. if re.search("/dev/tw|/dev/cciss/c|/dev/sg|/dev/sd", custom_options) is None: