vim: src/fileio.c comparison

comparison src/fileio.c @ 557:862863033fdd v7.0158

updated for version 7.0158

author	vimboss
date	Wed, 23 Nov 2005 21:25:05 +0000
parents	74c2ea3c3081
children	afd8e69dba19

comparison

equal deleted inserted replaced

-:f9eaf0a9872d
+:862863033fdd
 	     * - we can't set the owner/group of the new file
 	     */
 	    if (st_old.st_nlink > 1
 		    || mch_lstat((char *)fname, &st) < 0
 		    || st.st_dev != st_old.st_dev
-		    || st.st_ino != st_old.st_ino)
+		    || st.st_ino != st_old.st_ino
+#  ifndef HAVE_FCHOWN
+		    || st.st_uid != st_old.st_uid
+		    || st.st_gid != st_old.st_gid
+#  endif
+		    )
 		backup_copy = TRUE;
 	    else
 # endif
 	    {
 		/*
 		 */
 		STRCPY(IObuff, fname);
 		for (i = 4913; ; i += 123)
 		{
 		    sprintf((char *)gettail(IObuff), "%d", i);
-		    if (mch_stat((char *)IObuff, &st) < 0)
+		    if (mch_lstat((char *)IObuff, &st) < 0)
 			break;
 		}
-		fd = mch_open((char *)IObuff, O_CREAT|O_WRONLY|O_EXCL, perm);
+		fd = mch_open((char *)IObuff,
-		close(fd);
+				    O_CREAT|O_WRONLY|O_EXCL|O_NOFOLLOW, perm);
 		if (fd < 0)	/* can't write in directory */
 		    backup_copy = TRUE;
 		else
 		{
 # ifdef UNIX
-		    chown((char *)IObuff, st_old.st_uid, st_old.st_gid);
+#  ifdef HAVE_FCHOWN
-		    (void)mch_setperm(IObuff, perm);
+		    fchown(fd, st_old.st_uid, st_old.st_gid);
+#  endif
 		    if (mch_stat((char *)IObuff, &st) < 0
 			    || st.st_uid != st_old.st_uid
 			    || st.st_gid != st_old.st_gid
 			    || st.st_mode != perm)
 			backup_copy = TRUE;
 # endif
 		    mch_remove(IObuff);
+		    close(fd);
 		}
 	    }
 	}
 # ifdef UNIX
 		    /* remove old backup, if present */
 		    mch_remove(backup);
 		    /* Open with O_EXCL to avoid the file being created while
 		     * we were sleeping (symlink hacker attack?) */
 		    bfd = mch_open((char *)backup,
-				O_WRONLY|O_CREAT|O_EXTRA|O_EXCL, perm & 0777);
+				O_WRONLY|O_CREAT|O_EXTRA|O_EXCL|O_NOFOLLOW,
+								 perm & 0777);
 		    if (bfd < 0)
 		    {
 			vim_free(backup);
 			backup = NULL;
 		    }
 			 * Try to set the group of the backup same as the
 			 * original file. If this fails, set the protection
 			 * bits for the group same as the protection bits for
 			 * others.
 			 */
-			if (st_new.st_gid != st_old.st_gid &&
+			if (st_new.st_gid != st_old.st_gid
 # ifdef HAVE_FCHOWN  /* sequent-ptx lacks fchown() */
-				    fchown(bfd, (uid_t)-1, st_old.st_gid) != 0
+				&& fchown(bfd, (uid_t)-1, st_old.st_gid) != 0
-# else
-			  chown((char *)backup, (uid_t)-1, st_old.st_gid) != 0
 # endif
 						)
 			    mch_setperm(backup,
 					  (perm & 0707) | ((perm & 07) << 3));
 #endif
 	errmsg = (char_u *)_("E667: Fsync failed");
 	end = 0;
 }
 #endif
+#ifdef UNIX
+/* When creating a new file, set its owner/group to that of the original
+* file.  Get the new device and inode number. */
+if (backup != NULL && !backup_copy)
+{
+# ifdef HAVE_FCHOWN
+	struct stat	st;
+	/* don't change the owner when it's already OK, some systems remove
+	 * permission or ACL stuff */
+	if (mch_stat((char *)wfname, &st) < 0
+		|| st.st_uid != st_old.st_uid
+		|| st.st_gid != st_old.st_gid)
+	{
+	    fchown(fd, st_old.st_uid, st_old.st_gid);
+	    if (perm >= 0)	/* set permission again, may have changed */
+		(void)mch_setperm(wfname, perm);
+	}
+# endif
+	buf_setino(buf);
+}
+#endif
 if (close(fd) != 0)
 {
 	errmsg = (char_u *)_("E512: Close failed");
 	end = 0;
 }
 #ifdef HAVE_ACL
 /* Probably need to set the ACL before changing the user (can't set the
 * ACL on a file the user doesn't own). */
 if (!backup_copy)
 	mch_set_acl(wfname, acl);
-#endif
-#ifdef UNIX
-/* When creating a new file, set its owner/group to that of the original
-* file.  Get the new device and inode number. */
-if (backup != NULL && !backup_copy)
-{
-	struct stat	st;
-	/* don't change the owner when it's already OK, some systems remove
-	 * permission or ACL stuff */
-	if (mch_stat((char *)wfname, &st) < 0
-		|| st.st_uid != st_old.st_uid
-		|| st.st_gid != st_old.st_gid)
-	{
-	    chown((char *)wfname, st_old.st_uid, st_old.st_gid);
-	    if (perm >= 0)	/* set permission again, may have changed */
-		(void)mch_setperm(wfname, perm);
-	}
-	buf_setino(buf);
-}
 #endif
 #if defined(FEAT_MBYTE) && defined(FEAT_EVAL)
 if (wfname != fname)
 	else
 	{
 	    int empty_fd;
 	    if (org == NULL
-		    || (empty_fd = mch_open(org, O_CREAT | O_EXTRA | O_EXCL,
+		    || (empty_fd = mch_open(org,
+				      O_CREAT | O_EXTRA | O_EXCL | O_NOFOLLOW,
 					perm < 0 ? 0666 : (perm & 0777))) < 0)
 	      EMSG(_("E206: patchmode: can't touch empty original file"));
 	    else
 	      close(empty_fd);
 	}
 fd_in = mch_open((char *)from, O_RDONLY|O_EXTRA, 0);
 if (fd_in == -1)
 	return -1;
 /* Create the new file with same permissions as the original. */
-fd_out = mch_open((char *)to, O_CREAT|O_EXCL|O_WRONLY|O_EXTRA, (int)perm);
+fd_out = mch_open((char *)to,
+		       O_CREAT|O_EXCL|O_WRONLY|O_EXTRA|O_NOFOLLOW, (int)perm);
 if (fd_out == -1)
 {
 	close(fd_in);
 	return -1;
 }
 char_u	*save_ei;
 save_ei = vim_strsave(p_ei);
 if (save_ei != NULL)
 {
-	new_ei = vim_strnsave(p_ei, (int)STRLEN(p_ei) + 8);
+	new_ei = vim_strnsave(p_ei, (int)(STRLEN(p_ei) + STRLEN(what)));
 	if (new_ei != NULL)
 	{
 	    STRCAT(new_ei, what);
 	    set_string_option_direct((char_u *)"ei", -1, new_ei, OPT_FREE);
 	    vim_free(new_ei);

Mercurial > vim

comparison src/fileio.c @ 557:862863033fdd v7.0158