Prevent memcpy(nullptr, ..., 0) call (#4736)

bloomberg · May 13, 2024 · d766608 · d766608
1 parent 22fb795
commit d766608
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/groups/bdl/bdlbb/bdlbb_blobstreambuf.cpp b/groups/bdl/bdlbb/bdlbb_blobstreambuf.cpp
@@ -556,10 +556,13 @@ bsl::streamsize OutBlobStreamBuf::xsputn(const char_type *source,
         bsl::streamsize remainingChars = epptr() - pptr();
         bsl::streamsize canCopy        = bsl::min(remainingChars, numLeft);
 
-        bsl::memcpy(pptr(), source + numCopied, canCopy);
-        pbump(static_cast<int>(canCopy));
-        numCopied += canCopy;
-        numLeft -= canCopy;
+        if (canCopy > 0) {
+            BSLS_ASSERT(pptr());
+            bsl::memcpy(pptr(), source + numCopied, canCopy);
+            pbump(static_cast<int>(canCopy));
+            numCopied += canCopy;
+            numLeft -= canCopy;
+        }
 
         if (0 < numLeft) {
             if (traits_type::eof() ==