Remove unused and broken HIDPowerDevice_::sendDate method

The method currently uses a local bval variable as argument when calling HID().SendReport(...). This is problematic, since the SendReport method doesn't use bval immediately. It instead captures a pointer to bval which is accessed when the report is sent at a later point. This leads to a use-after-free situation when the pointer captured by SendReport no longer point to bval, but some other unknown data.

Author: forderud

src/HIDPowerDevice.cpp

@@ -250,11 +250,6 @@ void HIDPowerDevice_::setSerial(const char* s) {
 void HIDPowerDevice_::end(void) {
 }
 
-int HIDPowerDevice_::sendDate(uint16_t id, uint16_t year, uint8_t month, uint8_t day) {
-    uint16_t bval = (year - 1980)*512 + month * 32 + day;
-    return HID().SendReport(id, &bval, sizeof (bval));
-}
-
 int HIDPowerDevice_::sendReport(uint16_t id, const void* bval, int len) {
     return HID().SendReport(id, bval, len);
 }

src/HIDPowerDevice.h

@@ -121,7 +121,6 @@ class HIDPowerDevice_  {
 
   void end(void);
 
-  int sendDate(uint16_t id, uint16_t year, uint8_t month, uint8_t day);
   int sendReport(uint16_t id, const void* bval, int len);
 
   int setFeature(uint16_t id, const void* data, int len);