pw_transfer: Fix when to write Chunk::PendingBytes

Change-Id: Ie7ac64ef6ce5bfcade62fab93a7450e939a30727 Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/98521 Commit-Queue: Carlos Chinchilla <[email protected]> Reviewed-by: Alexei Frolov <[email protected]> Pigweed-Auto-Submit: Carlos Chinchilla <[email protected]>
google · Jun 17, 2022 · cf60b1c · cf60b1c
1 parent d7431f4
commit cf60b1c
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pw_transfer/chunk.cc b/pw_transfer/chunk.cc
@@ -207,7 +207,9 @@ Result<ConstByteSpan> Chunk::Encode(ByteSpan buffer) const {
 
     // In the legacy protocol, the pending_bytes field must be set alongside
     // window_end_offset, as some transfer implementations require it.
-    encoder.WritePendingBytes(window_end_offset_ - offset_).IgnoreError();
+    if (window_end_offset_ != 0) {
+      encoder.WritePendingBytes(window_end_offset_ - offset_).IgnoreError();
+    }
   }
 
   if (max_chunk_size_bytes_.has_value()) {