AP_Common: added return to ExpandingString append

libraries/AP_Common/ExpandingString.cpp

@@ -84,16 +84,19 @@ void ExpandingString::printf(const char *format, ...)
 /*
   print into the buffer, expanding if needed
  */
-void ExpandingString::append(const char *s, uint32_t len)
+bool ExpandingString::append(const char *s, uint32_t len)
 {
     if (allocation_failed) {
-        return;
+        return false;
     }
     if (buflen - used < len && !expand(len)) {
-        return;
+        return false;
+    }
+    if (s != nullptr) {
+        memcpy(&buf[used], s, len);
     }
-    memcpy(&buf[used], s, len);
     used += len;
+    return true;
 }
 
 ExpandingString::~ExpandingString()

libraries/AP_Common/ExpandingString.h

@@ -36,8 +36,8 @@ public:
     // print into the string
     void printf(const char *format, ...) FMT_PRINTF(2,3);
 
-    // append data to the string
+    // append data to the string. s can be null for zero fill
-    void append(const char *s, uint32_t len);
+    bool append(const char *s, uint32_t len);
 
     // destructor
     ~ExpandingString();