Commit Diff

Commit:
72f5a11aff98b21aafaf38400707411549e7c12b
From:
Omar Polo <op@omarpolo.com>
Via:
Thomas Adam <thomas@xteddy.org>
Date:
Message:
gotwebd: avoid needless double fseek() got_gotweb_closefile() already takes care of rewinding before truncating and closing, so no need to replicate here. ok stsp@
Actions:
Patch | Tree
commit - aa2aecab69ed4f6ff9ab2a5bb7e32109cf95aefc
commit + 72f5a11aff98b21aafaf38400707411549e7c12b
blob - 7a98944b00d81ef90ba04bd4b7c8ba9f024d2d59
blob + c9ec594eafc00789a813deeb3f864bf32d7398c4
--- gotwebd/got_operations.c
+++ gotwebd/got_operations.c
@@ -1228,17 +1228,7 @@ got_open_diff_for_output(FILE **fp, struct request *c)
 		error = got_error(GOT_ERR_OBJ_TYPE);
 	}
 	if (error)
-		goto done;
-
-	if (fseek(f1, 0, SEEK_SET) == -1) {
-		error = got_ferror(f1, GOT_ERR_IO);
-		goto done;
-	}
-
-	if (fseek(f2, 0, SEEK_SET) == -1) {
-		error = got_ferror(f2, GOT_ERR_IO);
 		goto done;
-	}
 
 	if (fseek(f3, 0, SEEK_SET) == -1) {
 		error = got_ferror(f3, GOT_ERR_IO);