Author: mjg
Date: Mon Oct  6 07:22:48 2014
New Revision: 272600
URL: https://svnweb.freebsd.org/changeset/base/272600

Log:
  devfs: tidy up after 272596
  
  This moves a var to an if statement, no functional changes.
  
  MFC after:    1 week

Modified:
  head/sys/fs/devfs/devfs_vnops.c

Modified: head/sys/fs/devfs/devfs_vnops.c
==============================================================================
--- head/sys/fs/devfs/devfs_vnops.c     Mon Oct  6 07:09:47 2014        
(r272599)
+++ head/sys/fs/devfs/devfs_vnops.c     Mon Oct  6 07:22:48 2014        
(r272600)
@@ -545,8 +545,8 @@ devfs_close(struct vop_close_args *ap)
         * if the reference count is 2 (this last descriptor
         * plus the session), release the reference from the session.
         */
-       oldvp = NULL;
        if (td && vp == td->td_proc->p_session->s_ttyvp) {
+               oldvp = NULL;
                sx_xlock(&proctree_lock);
                if (vp == td->td_proc->p_session->s_ttyvp) {
                        SESS_LOCK(td->td_proc->p_session);
@@ -561,9 +561,9 @@ devfs_close(struct vop_close_args *ap)
                        SESS_UNLOCK(td->td_proc->p_session);
                }
                sx_xunlock(&proctree_lock);
+               if (oldvp != NULL)
+                       vrele(oldvp);
        }
-       if (oldvp != NULL)
-               vrele(oldvp);
        /*
         * We do not want to really close the device if it
         * is still in use unless we are trying to close it
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to