Agreed-upon feedback updates.

Author: pbarry-r7

lib/rex/post/hwbridge/extensions/automotive/uds_errors.rb

@@ -56,8 +56,8 @@ module UDSErrors
   0x78 => "RCRRP",
   0x7E => "SFNSIAS",
   0x7F => "SNSIAS",
-  0x81 => "RTH",
-  0x82 => "RTL",
+  0x81 => "RPMTH",
+  0x82 => "RPMTL",
   0x83 => "EIR",
   0x84 => "EINR",
   0x85 => "ERTTL",

lib/rex/post/hwbridge/ui/console/command_dispatcher/automotive.rb

@@ -219,7 +219,7 @@ def cmd_isotpsend(*args)
     bytes = data.scan(/../)  # Break up data string into 2 char (byte) chunks
     if bytes.size > 8
       print_error("Data section can only contain a max of 8 bytes (for now)")
-      return result
+      return
     end
     opt = {}
     opt['TIMEOUT'] = timeout unless timeout.nil?
@@ -281,7 +281,15 @@ def cmd_testerpresent(*args)
       end
       return
     end
-    unless stop
+    if stop
+      if self.tpjobs[stopid]
+        self.tpjobs[stopid].kill
+        self.tpjobs[stopid] = nil
+        print_status("Stopped TesterPresent #{stopid}")
+      else
+        print_error("TesterPresent #{stopid} was not running")
+      end
+    else
       jid = self.tpjob_id
       print_status("Starting TesterPresent sender (#{self.tpjob_id})")
       self.tpjob_id += 1
@@ -300,14 +308,6 @@ def cmd_testerpresent(*args)
         self.tpjobs[myjid] = nil
         print_status("TesterPreset #{myjid} has stopped (#{::Thread.current[:args].inspect})")
       end
-    else
-      if self.tpjobs[stopid]
-        self.tpjobs[stopid].kill
-        self.tpjobs[stopid] = nil
-        print_status("Stopped TesterPresent #{stopid}")
-      else
-        print_error("TesterPresent #{stopid} was not running")
-      end
     end
   end
 

lib/rex/post/hwbridge/ui/console/command_dispatcher/core.rb

@@ -219,10 +219,10 @@ def cmd_status(*args)
     print_status("Device: #{status['device_name']}") if status.key? 'device_name'
     print_status("FW Version: #{status['fw_version']}") if status.key? 'fw_version'
     print_status("HW Version: #{status['hw_version']}") if status.key? 'hw_version'
-    print_status("Uptime: #{stats["uptime"]} seconds") if stats.key? "uptime"
-    print_status("Packets Sent: #{stats["packet_stats"]}") if stats.key? "packet_stats"
-    print_status("Last packet Sent: #{Time.at(stats["last_request"])}") if stats.key? "last_request"
-    print_status("Voltage: #{stats["voltage"]}") if stats.key? "voltage" and not stats["voltage"] == "not supported"
+    print_status("Uptime: #{stats['uptime']} seconds") if stats.key? 'uptime'
+    print_status("Packets Sent: #{stats['packet_stats']}") if stats.key? 'packet_stats'
+    print_status("Last packet Sent: #{Time.at(stats['last_request'])}") if stats.key? 'last_request'
+    print_status("Voltage: #{stats['voltage']}") if stats.key? 'voltage' and not stats['voltage'] == 'not supported'
   end
 
   def cmd_specialty_help

lib/rex/post/hwbridge/ui/console/command_dispatcher/rftransceiver.rb

@@ -290,7 +290,7 @@ def cmd_recv(*args)
     arg['blocksize'] = blocksize unless blocksize == -1
     arg['timeout'] = timeout unless timeout == -1
     r = client.rftransceiver.rfrecv(idx, arg)
-    if r.key?('data') && r.has_key?('timestamp')
+    if r.key?('data') && r.key?('timestamp')
       print_line(" #{r['timestamp']}: #{r['data'].inspect}")
     else
       print_line("Error")