Skip to content

Commit

Permalink
dist/tools/usb-serial: Fix handling of None while quoting
Browse files Browse the repository at this point in the history
This fixes:

    Traceback (most recent call last):
      File "/home/maribu/Repos/software/RIOT/master/dist/tools/usb-serial/ttys.py", line 259, in <module>
        print_ttys(sys.argv)
      File "/home/maribu/Repos/software/RIOT/master/dist/tools/usb-serial/ttys.py", line 255, in print_ttys
        print_results(args, ttys)
      File "/home/maribu/Repos/software/RIOT/master/dist/tools/usb-serial/ttys.py", line 189, in print_results
        if item.rfind(args.format_sep) >= 0:
           ^^^^^^^^^^
    AttributeError: 'NoneType' object has no attribute 'rfind'

Which occurs while testing whether a string requires special quoting
if an attribute is None.
  • Loading branch information
maribu committed Jun 21, 2023
1 parent 4deca3d commit 6192c62
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dist/tools/usb-serial/ttys.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ def print_results(args, ttys):
line = ""
for fmt in args.format:
item = tty[fmt]
if item.rfind(args.format_sep) >= 0:
if item is not None and item.rfind(args.format_sep) >= 0:
# item contains separator --> quote it
# using json.dumps to also escape quotation chars and other
# unsafe stuff
Expand Down

0 comments on commit 6192c62

Please sign in to comment.