Skip to content

Commit

Permalink
untested bytecast in smart.py rockstor#2564
Browse files Browse the repository at this point in the history
Custom options autodev related byte assertion.
  • Loading branch information
phillxnet committed Jun 6, 2023
1 parent 6edcdb1 commit c53966a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/rockstor/system/smart.py
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down

0 comments on commit c53966a

Please sign in to comment.