Allow for partial writes as suggested by @crog. Fixes #2.

CTrabant · Aug 3, 2015 · fbd76fb · fbd76fb
1 parent 94a53ec
commit fbd76fb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -1,8 +1,10 @@
-2015.8.2:
+2015.8.2: 1.1
 	- Rework deflate() logic for flexibility and simplicity.
 	- Add zs_entryflush() as a simple wrapper to flush internal compression
 	buffers without new data.  Thanks to @Crog for reporting and suggesting.
 	- Update zipfiles.c example code to use new zs_entryflush().
+	- Allow retry of write() in zs_writedata() for incomplete writes as
+	suggested by @Crog.
 	- Add Apache 2 licensing notices.
 
 2013.9.28: 1.0

diff --git a/fdzipstream.c b/fdzipstream.c
@@ -913,13 +913,13 @@ zs_writedata ( ZIPstream *zstream, unsigned char *writeentry,
 
       lwritestatus = write (zstream->fd, writeentry+written, writelen);
 
-      if ( lwritestatus != writelen )
+      if ( lwritestatus <= 0 )
 	{
 	  return lwritestatus;
 	}
 
-      zstream->WriteOffset += writelen;
-      written += writelen;
+      zstream->WriteOffset += lwritestatus;
+      written += lwritestatus;
     }
 
   return written;