isohybrid: Use ftruncate instead of seek for final padding
authorP J P <pj.pandit@yahoo.co.in>
Tue, 14 Sep 2010 06:38:37 +0000 (12:08 +0530)
committerH. Peter Anvin <hpa@zytor.com>
Wed, 13 Oct 2010 05:35:05 +0000 (22:35 -0700)
Pad the image via ftruncate instead of seeking to the end (which was
done incorrectly).

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
utils/isohybrid.c

index 57c1015..7ee9a7f 100644 (file)
@@ -543,15 +543,11 @@ main(int argc, char *argv[])
 
     if (padding)
     {
-        if (!(buf = realloc(buf, padding)))
-            err(1, "%s: could not re-size buffer", argv[0]);
+        if (fsync(fileno(fp)))
+            err(1, "%s: could not synchronise", argv[0]);
 
-        if (fseek(fp, isostat.st_size, SEEK_SET))
-            err(1, "%s: seek error - 6", argv[0]);
-
-        memset(buf, 0, padding);
-        if (fwrite(buf, sizeof(char), padding, fp) != (size_t)padding)
-            err(1, "%s: write error - 2", argv[0]);
+        if (ftruncate(fileno(fp), isostat.st_size + padding))
+            err(1, "%s: could not add padding bytes", argv[0]);
     }
 
     free(buf);