check chown() return value

Signed-off-by: Mike Frysinger <vapier@gmail.com>
vapier · Jan 29, 2010 · c3f07e9 · c3f07e9
1 parent e699c99
commit c3f07e9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/Changes b/Changes
@@ -1,3 +1,6 @@
+(N)compress version 4.2.4.3
+	o Check return value of chown().
+
 (N)compress version 4.2.4.2
 	o Integrate build/man touchups from Debian.
 	o Add fix from Fedora for handling of 0 byte files.

diff --git a/compress42.c b/compress42.c
@@ -1236,7 +1236,12 @@ comprexx(fileptr)
 						exit_code = 1;
 					}
 #ifndef	DOS
-					chown(ofname, infstat.st_uid, infstat.st_gid);	/* Copy ownership */
+					if (chown(ofname, infstat.st_uid, infstat.st_gid))	/* Copy ownership */
+					{
+						fprintf(stderr, "\nchown error (ignored) ");
+						perror(ofname);
+						exit_code = 1;
+					}
 #endif
 #endif
 					remove_ofname = 0;