ArduPilot / MethodicConfigurator / 27105673598

94%

push

github

fix(MAVFTP): prevent MAVFTP write path crashes for empty files and None write_list

- Fix wrong condition order in __send_more_writes (line 875):
  len(None) raises TypeError before is None check runs.
  Swap to: if self.write_list is None or len(self.write_list) == 0

- Fix ZeroDivisionError in __handle_write_reply (line 915):
  Uploading empty file sets write_total=0, causing modulo by zero.
  Early-return with ERR_None when write_total == 0.

- Fix missing None guard before len(write_list) in __send_more_writes
  (line 886): same TypeError if write_list is None at that point.

Add three regression tests covering each crash scenario.

1948 of 2020 branches covered (96.44%)

12420 of 13151 relevant lines covered (94.44%)

2.83 hits per line