fixing warnings about ignoring return value

Author: gruszczy

structure/necessary.c

@@ -379,7 +379,8 @@ char * build_snapshot(struct file_system_info *fsinfo, revision_t *revisions, in
             gmstrcpy(&snapshot, data_dir, "/", fsinfo->repo_names[repo_index], "/", revisions[i].file, 0);  
         if ((revision_desc = open(snapshot, O_RDONLY)) == -1)
             build_snapshot_error;
-        write(snapshot_desc, "\n", 1);
+        if (write(snapshot_desc, "\n", 1) == -1)
+            build_snapshot_error;
         if (gdesccopy(revision_desc, snapshot_desc))
             build_snapshot_error;
         gstrdel(snapshot);

support/grdiff.c

@@ -329,10 +329,14 @@ int snapshot_copy(char *revision, char *target, char *directory){
 		return -1;
 	};
 	while ((result = read(revision_desc, buffer, 1024)) != 0)
-		write(snapshot_desc, buffer, result);
+		if (write(snapshot_desc, buffer, result) == -1){
+            close(snapshot_desc);
+            close(revision_desc);
+            return -1;
+        }
 	close(snapshot_desc);
 	close(revision_desc);
-	return 0;
+	return result;
 
 };
 
@@ -355,12 +359,20 @@ int snapshot_append(char *revision, char *target, char *directory){
 		return -1;
 	};
 	buffer[0] = '\n';
-	write(snapshot_desc, buffer, 1);
+	if (write(snapshot_desc, buffer, 1) == -1){
+        close(snapshot_desc);
+        return -1;
+    }
+        
 	while ((result = read(revision_desc, buffer, 1024)) != 0)
-		write(snapshot_desc, buffer, result);
+		if (write(snapshot_desc, buffer, result) == -1){
+            close(snapshot_desc);
+            close(revision_desc);
+            return -1;
+        }
 	close(snapshot_desc);
 	close(revision_desc);
-	return 0;
+	return result;
 
 };
 

support/gutils.c

@@ -6,7 +6,8 @@ int gdesccopy(int source, int dest){
 	char buffer[1024];
     int result = 0;
 	while ((result = read(source, buffer, 1024)) > 0){
-		write(dest, buffer, result);
+		if (write(dest, buffer, result) == -1)
+            return -1;
     }
     if (result == -1)
         return -1;