]> git.saurik.com Git - apple/xnu.git/blobdiff - bsd/kern/posix_sem.c
xnu-6153.141.1.tar.gz
[apple/xnu.git] / bsd / kern / posix_sem.c
index 54b92f059d6a65be2d31ae788cdd49a90fab3cb3..b1119d15381f0860b6e3dea3c44e2a56d45ee342 100644 (file)
@@ -2,7 +2,7 @@
  * Copyright (c) 2000-2007 Apple Inc. All rights reserved.
  *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_START@
- * 
+ *
  * This file contains Original Code and/or Modifications of Original Code
  * as defined in and that are subject to the Apple Public Source License
  * Version 2.0 (the 'License'). You may not use this file except in
  * unlawful or unlicensed copies of an Apple operating system, or to
  * circumvent, violate, or enable the circumvention or violation of, any
  * terms of an Apple operating system software license agreement.
- * 
+ *
  * Please obtain a copy of the License at
  * http://www.opensource.apple.com/apsl/ and read it before using this file.
- * 
+ *
  * The Original Code and all software distributed under the License are
  * distributed on an 'AS IS' basis, WITHOUT WARRANTY OF ANY KIND, EITHER
  * EXPRESS OR IMPLIED, AND APPLE HEREBY DISCLAIMS ALL SUCH WARRANTIES,
@@ -22,7 +22,7 @@
  * FITNESS FOR A PARTICULAR PURPOSE, QUIET ENJOYMENT OR NON-INFRINGEMENT.
  * Please see the License for the specific language governing rights and
  * limitations under the License.
- * 
+ *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_END@
  */
 /*
 #define f_ops f_fglob->fg_ops
 #define f_offset f_fglob->fg_offset
 #define f_data f_fglob->fg_data
-#define        PSEMNAMLEN      31      /* maximum name segment length we bother with */
+#define PSEMNAMLEN      31      /* maximum name segment length we bother with */
 
 struct pseminfo {
-       unsigned int    psem_flags;
-       unsigned int    psem_usecount;
-       mode_t          psem_mode;
-       uid_t           psem_uid;
-       gid_t           psem_gid;
-       char            psem_name[PSEMNAMLEN + 1];      /* segment name */
-       semaphore_t     psem_semobject;
-       struct label *  psem_label;
-       pid_t           psem_creator_pid;
-       uint64_t        psem_creator_uniqueid;
+       unsigned int    psem_flags;
+       unsigned int    psem_usecount;
+       mode_t          psem_mode;
+       uid_t           psem_uid;
+       gid_t           psem_gid;
+       char            psem_name[PSEMNAMLEN + 1];      /* segment name */
+       semaphore_t     psem_semobject;
+       struct label *  psem_label;
+       pid_t           psem_creator_pid;
+       uint64_t        psem_creator_uniqueid;
 };
 #define PSEMINFO_NULL (struct pseminfo *)0
 
-#define        PSEM_NONE       1
-#define        PSEM_DEFINED    2
-#define        PSEM_ALLOCATED  4
-#define        PSEM_MAPPED     8
-#define        PSEM_INUSE      0x10
-#define        PSEM_REMOVED    0x20
-#define        PSEM_INCREATE   0x40
-#define        PSEM_INDELETE   0x80
-
-struct psemcache {
-       LIST_ENTRY(psemcache) psem_hash;        /* hash chain */
-       struct  pseminfo *pseminfo;             /* vnode the name refers to */
-       int     psem_nlen;              /* length of name */
-       char    psem_name[PSEMNAMLEN + 1];      /* segment name */
+#define PSEM_NONE       1
+#define PSEM_DEFINED    2
+#define PSEM_ALLOCATED  4
+#define PSEM_MAPPED     8
+#define PSEM_INUSE      0x10
+#define PSEM_REMOVED    0x20
+#define PSEM_INCREATE   0x40
+#define PSEM_INDELETE   0x80
+
+struct  psemcache {
+       LIST_ENTRY(psemcache) psem_hash;        /* hash chain */
+       struct  pseminfo *pseminfo;             /* vnode the name refers to */
+       int     psem_nlen;              /* length of name */
+       char    psem_name[PSEMNAMLEN + 1];      /* segment name */
 };
 #define PSEMCACHE_NULL (struct psemcache *)0
 
@@ -128,19 +128,19 @@ struct    psemcache {
 #define PSEMCACHE_FOUND    (-1)
 #define PSEMCACHE_NEGATIVE (ENOENT)
 
-struct psemstats {
-       long    goodhits;               /* hits that we can really use */
-       long    neghits;                /* negative hits that we can use */
-       long    badhits;                /* hits we must drop */
-       long    falsehits;              /* hits with id mismatch */
-       long    miss;           /* misses */
-       long    longnames;              /* long names that ignore cache */
+struct  psemstats {
+       long    goodhits;               /* hits that we can really use */
+       long    neghits;                /* negative hits that we can use */
+       long    badhits;                /* hits we must drop */
+       long    falsehits;              /* hits with id mismatch */
+       long    miss;           /* misses */
+       long    longnames;              /* long names that ignore cache */
 };
 
 struct psemname {
-       char    *psem_nameptr;  /* pointer to looked up name */
-       long    psem_namelen;   /* length of looked up component */
-       u_int32_t       psem_hash;      /* hash value of looked up name */
+       char    *psem_nameptr;  /* pointer to looked up name */
+       long    psem_namelen;   /* length of looked up component */
+       u_int32_t       psem_hash;      /* hash value of looked up name */
 };
 
 struct psemnode {
@@ -155,43 +155,34 @@ struct psemnode {
 
 #define PSEMHASH(pnp) \
        (&psemhashtbl[(pnp)->psem_hash & psemhash])
-LIST_HEAD(psemhashhead, psemcache) *psemhashtbl;       /* Hash Table */
-u_long psemhash;                               /* size of hash table - 1 */
-long   psemnument;                     /* number of cache entries allocated */
-long   posix_sem_max = 10000;          /* tunable for max POSIX semaphores */
-                                       /* 10000 limits to ~1M of memory */
+LIST_HEAD(psemhashhead, psemcache) * psemhashtbl;        /* Hash Table */
+u_long  psemhash;                               /* size of hash table - 1 */
+long    psemnument;                     /* number of cache entries allocated */
+long    posix_sem_max = 10000;          /* tunable for max POSIX semaphores */
+                                        /* 10000 limits to ~1M of memory */
 SYSCTL_NODE(_kern, KERN_POSIX, posix, CTLFLAG_RW | CTLFLAG_LOCKED, 0, "Posix");
 SYSCTL_NODE(_kern_posix, OID_AUTO, sem, CTLFLAG_RW | CTLFLAG_LOCKED, 0, "Semaphores");
-SYSCTL_LONG (_kern_posix_sem, OID_AUTO, max, CTLFLAG_RW | CTLFLAG_LOCKED, &posix_sem_max, "max");
+SYSCTL_LONG(_kern_posix_sem, OID_AUTO, max, CTLFLAG_RW | CTLFLAG_LOCKED, &posix_sem_max, "max");
 
-struct psemstats psemstats;            /* cache effectiveness statistics */
+struct psemstats psemstats;             /* cache effectiveness statistics */
 
 static int psem_access(struct pseminfo *pinfo, int mode, kauth_cred_t cred);
 static int psem_cache_search(struct pseminfo **,
-                               struct psemname *, struct psemcache **);
+    struct psemname *, struct psemcache **);
 static int psem_delete(struct pseminfo * pinfo);
 
-static int psem_read (struct fileproc *fp, struct uio *uio,
-                           int flags, vfs_context_t ctx);
-static int psem_write (struct fileproc *fp, struct uio *uio,
-                           int flags, vfs_context_t ctx);
-static int psem_ioctl (struct fileproc *fp, u_long com,
-                           caddr_t data, vfs_context_t ctx);
-static int psem_select (struct fileproc *fp, int which, void *wql, vfs_context_t ctx);
-static int psem_closefile (struct fileglob *fp, vfs_context_t ctx);
+static int psem_closefile(struct fileglob *fp, vfs_context_t ctx);
 static int psem_unlink_internal(struct pseminfo *pinfo, struct psemcache *pcache);
 
-static int psem_kqfilter (struct fileproc *fp, struct knote *kn, vfs_context_t ctx);
-
 static const struct fileops psemops = {
-       .fo_type = DTYPE_PSXSEM,
-       .fo_read = psem_read,
-       .fo_write = psem_write,
-       .fo_ioctl = psem_ioctl,
-       .fo_select = psem_select,
-       .fo_close = psem_closefile,
-       .fo_kqfilter = psem_kqfilter,
-       .fo_drain = NULL,
+       .fo_type     = DTYPE_PSXSEM,
+       .fo_read     = fo_no_read,
+       .fo_write    = fo_no_write,
+       .fo_ioctl    = fo_no_ioctl,
+       .fo_select   = fo_no_select,
+       .fo_close    = psem_closefile,
+       .fo_drain    = fo_no_drain,
+       .fo_kqfilter = fo_no_kqfilter,
 };
 
 static lck_grp_t       *psx_sem_subsys_lck_grp;
@@ -213,19 +204,18 @@ int psem_cache_purge_all(proc_t);
 __private_extern__ void
 psem_lock_init( void )
 {
+       psx_sem_subsys_lck_grp_attr = lck_grp_attr_alloc_init();
 
-    psx_sem_subsys_lck_grp_attr = lck_grp_attr_alloc_init();
-
-    psx_sem_subsys_lck_grp = lck_grp_alloc_init("posix shared memory", psx_sem_subsys_lck_grp_attr);
+       psx_sem_subsys_lck_grp = lck_grp_alloc_init("posix shared memory", psx_sem_subsys_lck_grp_attr);
 
-    psx_sem_subsys_lck_attr = lck_attr_alloc_init();
-    lck_mtx_init(& psx_sem_subsys_mutex, psx_sem_subsys_lck_grp, psx_sem_subsys_lck_attr);
+       psx_sem_subsys_lck_attr = lck_attr_alloc_init();
+       lck_mtx_init(&psx_sem_subsys_mutex, psx_sem_subsys_lck_grp, psx_sem_subsys_lck_attr);
 }
 
 /*
- * Lookup an entry in the cache 
- * 
- * 
+ * Lookup an entry in the cache
+ *
+ *
  * status of -1 is returned if matches
  * If the lookup determines that the name does not exist
  * (negative cacheing), a status of ENOENT is returned. If the lookup
@@ -234,7 +224,7 @@ psem_lock_init( void )
 
 static int
 psem_cache_search(struct pseminfo **psemp, struct psemname *pnp,
-                 struct psemcache **pcache)
+    struct psemcache **pcache)
 {
        struct psemcache *pcp, *nnp;
        struct psemhashhead *pcpp;
@@ -248,8 +238,9 @@ psem_cache_search(struct pseminfo **psemp, struct psemname *pnp,
        for (pcp = pcpp->lh_first; pcp != 0; pcp = nnp) {
                nnp = pcp->psem_hash.le_next;
                if (pcp->psem_nlen == pnp->psem_namelen &&
-                   !bcmp(pcp->psem_name, pnp->psem_nameptr,                                            (u_int)pcp-> psem_nlen))
+                   !bcmp(pcp->psem_name, pnp->psem_nameptr, (u_int)pcp->psem_nlen)) {
                        break;
+               }
        }
 
        if (pcp == 0) {
@@ -258,7 +249,7 @@ psem_cache_search(struct pseminfo **psemp, struct psemname *pnp,
        }
 
        /* We found a "positive" match, return the vnode */
-        if (pcp->pseminfo) {
+       if (pcp->pseminfo) {
                psemstats.goodhits++;
                /* TOUCH(ncp); */
                *psemp = pcp->pseminfo;
@@ -285,8 +276,9 @@ psem_cache_add(struct pseminfo *psemp, struct psemname *pnp, struct psemcache *p
        struct psemcache *dpcp;
 
 #if DIAGNOSTIC
-       if (pnp->psem_namelen > PSEMNAMLEN)
+       if (pnp->psem_namelen > PSEMNAMLEN) {
                panic("cache_enter: name too long");
+       }
 #endif
 
 
@@ -294,8 +286,9 @@ psem_cache_add(struct pseminfo *psemp, struct psemname *pnp, struct psemcache *p
        if (psem_cache_search(&dpinfo, pnp, &dpcp) == PSEMCACHE_FOUND) {
                return EEXIST;
        }
-       if (psemnument >= posix_sem_max)
+       if (psemnument >= posix_sem_max) {
                return ENOSPC;
+       }
        psemnument++;
        /*
         * Fill in cache info, if vp is NULL this is a "negative" cache entry.
@@ -311,9 +304,11 @@ psem_cache_add(struct pseminfo *psemp, struct psemname *pnp, struct psemcache *p
        {
                struct psemcache *p;
 
-               for (p = pcpp->lh_first; p != 0; p = p->psem_hash.le_next)
-                       if (p == pcp)
+               for (p = pcpp->lh_first; p != 0; p = p->psem_hash.le_next) {
+                       if (p == pcp) {
                                panic("psem:cache_enter duplicate");
+                       }
+               }
        }
 #endif
        LIST_INSERT_HEAD(pcpp, pcp, psem_hash);
@@ -333,13 +328,15 @@ static void
 psem_cache_delete(struct psemcache *pcp)
 {
 #if DIAGNOSTIC
-       if (pcp->psem_hash.le_prev == 0)
+       if (pcp->psem_hash.le_prev == 0) {
                panic("psem namecache purge le_prev");
-       if (pcp->psem_hash.le_next == pcp)
+       }
+       if (pcp->psem_hash.le_next == pcp) {
                panic("namecache purge le_next");
+       }
 #endif /* DIAGNOSTIC */
        LIST_REMOVE(pcp, psem_hash);
-       pcp->psem_hash.le_prev = NULL;  
+       pcp->psem_hash.le_prev = NULL;
        psemnument--;
 }
 
@@ -355,8 +352,9 @@ psem_cache_purge_all(__unused proc_t p)
        struct psemhashhead *pcpp;
        int error = 0;
 
-       if (kauth_cred_issuser(kauth_cred_get()) == 0)
+       if (kauth_cred_issuser(kauth_cred_get()) == 0) {
                return EPERM;
+       }
 
        PSEM_SUBSYS_LOCK();
        for (pcpp = &psemhashtbl[psemhash]; pcpp >= psemhashtbl; pcpp--) {
@@ -366,8 +364,9 @@ psem_cache_purge_all(__unused proc_t p)
                         * unconditionally unlink the cache entry
                         */
                        error = psem_unlink_internal(pcp->pseminfo, pcp);
-                       if (error)
+                       if (error) {
                                goto out;
+                       }
                }
        }
        assert(psemnument == 0);
@@ -375,9 +374,10 @@ psem_cache_purge_all(__unused proc_t p)
 out:
        PSEM_SUBSYS_UNLOCK();
 
-       if (error)
+       if (error) {
                printf("%s: Error %d removing all semaphores: %ld remain!\n",
-                      __func__, error, psemnument);
+                   __func__, error, psemnument);
+       }
        return error;
 }
 
@@ -396,13 +396,13 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
        char * nameptr;
        char * cp;
        size_t pathlen, plen;
-       int fmode ;
+       int fmode;
        int cmode = uap->mode;
        int value = uap->value;
        int incache = 0;
        struct psemcache *pcp = PSEMCACHE_NULL;
-       kern_return_t kret = KERN_INVALID_ADDRESS;      /* default fail */
-       
+       kern_return_t kret = KERN_INVALID_ADDRESS;      /* default fail */
+
        AUDIT_ARG(fflags, uap->oflag);
        AUDIT_ARG(mode, uap->mode);
        AUDIT_ARG(value32, uap->value);
@@ -413,7 +413,7 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
         * Preallocate everything we might need up front to avoid taking
         * and dropping the lock, opening us up to race conditions.
         */
-       MALLOC_ZONE(pnbuf, caddr_t, MAXPATHLEN, M_NAMEI, M_WAITOK);
+       MALLOC_ZONE(pnbuf, caddr_t, MAXPATHLEN, M_NAMEI, M_WAITOK | M_ZERO);
        if (pnbuf == NULL) {
                error = ENOSPC;
                goto bad;
@@ -425,7 +425,7 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
                goto bad;
        }
        AUDIT_ARG(text, pnbuf);
-       if ( (pathlen > PSEMNAMLEN) ) {
+       if ((pathlen > PSEMNAMLEN)) {
                error = ENAMETOOLONG;
                goto bad;
        }
@@ -438,7 +438,7 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
                        error = EINVAL;
                        goto bad;
                }
-        } else {
+       } else {
                error = EINVAL;
                goto bad;
        }
@@ -450,8 +450,8 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
        nd.psem_namelen = plen;
        nd.psem_hash = 0;
 
-        for (cp = nameptr, i=1; *cp != 0 && i <= plen; i++, cp++) {
-               nd.psem_hash += (unsigned char)*cp * i;
+       for (cp = nameptr, i = 1; *cp != 0 && i <= plen; i++, cp++) {
+               nd.psem_hash += (unsigned char)*cp * i;
        }
 
        /*
@@ -459,21 +459,22 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
         * left unmodified (NULL).
         */
        error = falloc(p, &fp, &indx, vfs_context_current());
-       if (error)
+       if (error) {
                goto bad;
+       }
 
        /*
         * We allocate a new entry if we are less than the maximum
         * allowed and the one at the front of the LRU list is in use.
         * Otherwise we use the one at the front of the LRU list.
         */
-       MALLOC(pcp, struct psemcache *, sizeof(struct psemcache), M_SHM, M_WAITOK|M_ZERO);
+       MALLOC(pcp, struct psemcache *, sizeof(struct psemcache), M_SHM, M_WAITOK | M_ZERO);
        if (pcp == PSEMCACHE_NULL) {
                error = ENOMEM;
                goto bad;
        }
 
-       MALLOC(new_pinfo, struct pseminfo *, sizeof(struct pseminfo), M_SHM, M_WAITOK|M_ZERO);
+       MALLOC(new_pinfo, struct pseminfo *, sizeof(struct pseminfo), M_SHM, M_WAITOK | M_ZERO);
        if (new_pinfo == NULL) {
                error = ENOSPC;
                goto bad;
@@ -488,34 +489,33 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
         * signal success or failure, which is why we set its default value
         * to KERN_INVALID_ADDRESS, above.
         */
-       
+
        fmode = FFLAGS(uap->oflag);
-       
-       if((fmode & O_CREAT)) {
-               
-               if((value < 0) || (value > SEM_VALUE_MAX)) {
+
+       if ((fmode & O_CREAT)) {
+               if ((value < 0) || (value > SEM_VALUE_MAX)) {
                        error = EINVAL;
                        goto bad;
                }
-               
+
                kret = semaphore_create(kernel_task, &new_pinfo->psem_semobject, SYNC_POLICY_FIFO, value);
-       
+
                if (kret != KERN_SUCCESS) {
                        switch (kret) {
-                               case KERN_RESOURCE_SHORTAGE:
-                                       error = ENOMEM;
-                                       break;
-                               case KERN_PROTECTION_FAILURE:
-                                       error = EACCES;
-                                       break;
-                               default:
-                                       error = EINVAL;
+                       case KERN_RESOURCE_SHORTAGE:
+                               error = ENOMEM;
+                               break;
+                       case KERN_PROTECTION_FAILURE:
+                               error = EACCES;
+                               break;
+                       default:
+                               error = EINVAL;
                        }
                        goto bad;
                }
        }
-       
-       MALLOC(new_pnode, struct psemnode *, sizeof(struct psemnode), M_SHM, M_WAITOK|M_ZERO);
+
+       MALLOC(new_pnode, struct psemnode *, sizeof(struct psemnode), M_SHM, M_WAITOK | M_ZERO);
        if (new_pnode == NULL) {
                error = ENOSPC;
                goto bad;
@@ -529,30 +529,31 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
                goto bad_locked;
        }
 
-       if (error == PSEMCACHE_FOUND)
+       if (error == PSEMCACHE_FOUND) {
                incache = 1;
-       else
+       } else {
                incache = 0;
+       }
 
        cmode &=  ALLPERMS;
 
-       if (((fmode & (O_CREAT | O_EXCL))==(O_CREAT | O_EXCL)) &&  incache) {
+       if (((fmode & (O_CREAT | O_EXCL)) == (O_CREAT | O_EXCL)) && incache) {
                /* sem exists and opened O_EXCL */
 #if notyet
                if (pinfo->psem_flags & PSEM_INDELETE) {
                }
-#endif 
+#endif
                AUDIT_ARG(posix_ipc_perm, pinfo->psem_uid,
-                       pinfo->psem_gid, pinfo->psem_mode);
+                   pinfo->psem_gid, pinfo->psem_mode);
                error = EEXIST;
                goto bad_locked;
        }
-       if (((fmode & (O_CREAT | O_EXCL))== O_CREAT) &&  incache) {
+       if (((fmode & (O_CREAT | O_EXCL)) == O_CREAT) && incache) {
                /* As per POSIX, O_CREAT has no effect */
                fmode &= ~O_CREAT;
        }
 
-       if ( (fmode & O_CREAT) ) {
+       if ((fmode & O_CREAT)) {
                /* create a new one (commit the allocation) */
                pinfo = new_pinfo;
                pinfo->psem_flags = PSEM_DEFINED | PSEM_INCREATE;
@@ -561,12 +562,12 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
                pinfo->psem_uid = kauth_getuid();
                pinfo->psem_gid = kauth_getgid();
                bcopy(pnbuf, &pinfo->psem_name[0], PSEMNAMLEN);
-               pinfo->psem_name[PSEMNAMLEN]= 0;
+               pinfo->psem_name[PSEMNAMLEN] = 0;
                pinfo->psem_flags &= ~PSEM_DEFINED;
                pinfo->psem_flags |= PSEM_ALLOCATED;
                pinfo->psem_creator_pid = p->p_pid;
                pinfo->psem_creator_uniqueid = p->p_uniqueid;
-               
+
 #if CONFIG_MACF
                error = mac_posixsem_check_create(kauth_cred_get(), nameptr);
                if (error) {
@@ -580,26 +581,26 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
                        error = ENOENT;
                        goto bad_locked;
                }
-               ifpinfo->psem_flags & PSEM_INDELETE) {
+               if (pinfo->psem_flags & PSEM_INDELETE) {
                        error = ENOENT;
                        goto bad_locked;
-               }       
+               }
                AUDIT_ARG(posix_ipc_perm, pinfo->psem_uid,
-                       pinfo->psem_gid, pinfo->psem_mode);
+                   pinfo->psem_gid, pinfo->psem_mode);
 #if CONFIG_MACF
                error = mac_posixsem_check_open(kauth_cred_get(), pinfo);
                if (error) {
                        goto bad_locked;
                }
 #endif
-               if ( (error = psem_access(pinfo, fmode, kauth_cred_get())) ) {
+               if ((error = psem_access(pinfo, fmode, kauth_cred_get()))) {
                        goto bad_locked;
                }
        }
 
        if (!incache) {
                /* if successful, this will consume the pcp */
-               if ( (error = psem_cache_add(pinfo, &nd, pcp)) ) {
+               if ((error = psem_cache_add(pinfo, &nd, pcp))) {
                        goto bad_locked;
                }
        }
@@ -636,19 +637,22 @@ sem_open(proc_t p, struct sem_open_args *uap, user_addr_t *retval)
 
        *retval = CAST_USER_ADDR_T(indx);
        FREE_ZONE(pnbuf, MAXPATHLEN, M_NAMEI);
-       return (0);
+       return 0;
 
 bad_locked:
        PSEM_SUBSYS_UNLOCK();
 bad:
-       if (pcp != PSEMCACHE_NULL)
+       if (pcp != PSEMCACHE_NULL) {
                FREE(pcp, M_SHM);
+       }
 
-       if (new_pnode != PSEMNODE_NULL)
+       if (new_pnode != PSEMNODE_NULL) {
                FREE(new_pnode, M_SHM);
+       }
 
-       if (fp != NULL)
+       if (fp != NULL) {
                fp_free(p, indx, fp);
+       }
 
        if (new_pinfo != PSEMINFO_NULL) {
                /*
@@ -666,9 +670,10 @@ bad:
                FREE(new_pinfo, M_SHM);
        }
 
-       if (pnbuf != NULL)
+       if (pnbuf != NULL) {
                FREE_ZONE(pnbuf, MAXPATHLEN, M_NAMEI);
-       return (error);
+       }
+       return error;
 }
 
 /*
@@ -678,13 +683,14 @@ static int
 psem_access(struct pseminfo *pinfo, int mode, kauth_cred_t cred)
 {
        int mode_req = ((mode & FREAD) ? S_IRUSR : 0) |
-                      ((mode & FWRITE) ? S_IWUSR : 0);
+           ((mode & FWRITE) ? S_IWUSR : 0);
 
        /* Otherwise, user id 0 always gets access. */
-       if (!suser(cred, NULL))
-               return (0);
+       if (!suser(cred, NULL)) {
+               return 0;
+       }
 
-       return(posix_cred_access(cred, pinfo->psem_uid, pinfo->psem_gid, pinfo->psem_mode, mode_req));
+       return posix_cred_access(cred, pinfo->psem_uid, pinfo->psem_gid, pinfo->psem_mode, mode_req);
 }
 
 static int
@@ -692,24 +698,27 @@ psem_unlink_internal(struct pseminfo *pinfo, struct psemcache *pcache)
 {
        PSEM_SUBSYS_ASSERT_HELD();
 
-       if (!pinfo || !pcache)
+       if (!pinfo || !pcache) {
                return EINVAL;
+       }
 
-       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED)) == 0)
+       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED)) == 0) {
                return EINVAL;
+       }
 
-       if (pinfo->psem_flags & PSEM_INDELETE)
+       if (pinfo->psem_flags & PSEM_INDELETE) {
                return 0;
+       }
 
        AUDIT_ARG(posix_ipc_perm, pinfo->psem_uid, pinfo->psem_gid,
-                 pinfo->psem_mode);
+           pinfo->psem_mode);
 
        pinfo->psem_flags |= PSEM_INDELETE;
        pinfo->psem_usecount--;
 
        if (!pinfo->psem_usecount) {
                psem_delete(pinfo);
-               FREE(pinfo,M_SHM);
+               FREE(pinfo, M_SHM);
        } else {
                pinfo->psem_flags |= PSEM_REMOVED;
        }
@@ -724,7 +733,7 @@ int
 sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *retval)
 {
        size_t i;
-       int error=0;
+       int error = 0;
        struct psemname nd;
        struct pseminfo *pinfo;
        char * nameptr;
@@ -737,7 +746,7 @@ sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *ret
 
        MALLOC_ZONE(pnbuf, caddr_t, MAXPATHLEN, M_NAMEI, M_WAITOK);
        if (pnbuf == NULL) {
-               return(ENOSPC);         /* XXX non-standard */
+               return ENOSPC;         /* XXX non-standard */
        }
        pathlen = MAXPATHLEN;
        error = copyinstr(uap->name, pnbuf, MAXPATHLEN, &pathlen);
@@ -759,7 +768,7 @@ sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *ret
                        error = EINVAL;
                        goto bad;
                }
-        } else {
+       } else {
                error = EINVAL;
                goto bad;
        }
@@ -767,10 +776,10 @@ sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *ret
 
        nd.psem_nameptr = nameptr;
        nd.psem_namelen = pathlen;
-       nd. psem_hash =0;
+       nd.psem_hash = 0;
 
-        for (cp = nameptr, i=1; *cp != 0 && i <= pathlen; i++, cp++) {
-               nd.psem_hash += (unsigned char)*cp * i;
+       for (cp = nameptr, i = 1; *cp != 0 && i <= pathlen; i++, cp++) {
+               nd.psem_hash += (unsigned char)*cp * i;
        }
 
        PSEM_SUBSYS_LOCK();
@@ -778,9 +787,8 @@ sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *ret
 
        if (error != PSEMCACHE_FOUND) {
                PSEM_SUBSYS_UNLOCK();
-               error = EINVAL;
+               error = ENOENT;
                goto bad;
-
        }
 
 #if CONFIG_MACF
@@ -790,7 +798,7 @@ sem_unlink(__unused proc_t p, struct sem_unlink_args *uap, __unused int32_t *ret
                goto bad;
        }
 #endif
-       if ( (error = psem_access(pinfo, pinfo->psem_mode, kauth_cred_get())) ) {
+       if ((error = psem_access(pinfo, pinfo->psem_mode, kauth_cred_get()))) {
                PSEM_SUBSYS_UNLOCK();
                goto bad;
        }
@@ -806,48 +814,56 @@ bad:
 int
 sem_close(proc_t p, struct sem_close_args *uap, __unused int32_t *retval)
 {
-       int fd = CAST_DOWN_EXPLICIT(int,uap->sem);
+       int fd = CAST_DOWN_EXPLICIT(int, uap->sem);
        struct fileproc *fp;
        int error = 0;
 
        AUDIT_ARG(fd, fd); /* XXX This seems wrong; uap->sem is a pointer */
 
        proc_fdlock(p);
-       error = fp_lookup(p,fd, &fp, 1);
+       error = fp_lookup(p, fd, &fp, 1);
        if (error) {
                proc_fdunlock(p);
-               return(error);
+               return error;
+       }
+       if (fp->f_type != DTYPE_PSXSEM) {
+               fp_drop(p, fd, fp, 1);
+               proc_fdunlock(p);
+               return EBADF;
        }
        procfdtbl_markclosefd(p, fd);
+       /* release the ref returned from fp_lookup before calling drain */
+       (void) os_ref_release_locked(&fp->f_iocount);
        fileproc_drain(p, fp);
        fdrelse(p, fd);
        error = closef_locked(fp, fp->f_fglob, p);
        fileproc_free(fp);
        proc_fdunlock(p);
-       return(error);
+       return error;
 }
 
 int
 sem_wait(proc_t p, struct sem_wait_args *uap, int32_t *retval)
 {
        __pthread_testcancel(1);
-       return(sem_wait_nocancel(p, (struct sem_wait_nocancel_args *)uap, retval));
+       return sem_wait_nocancel(p, (struct sem_wait_nocancel_args *)uap, retval);
 }
 
 int
 sem_wait_nocancel(proc_t p, struct sem_wait_nocancel_args *uap, __unused int32_t *retval)
 {
-       int fd = CAST_DOWN_EXPLICIT(int,uap->sem);
+       int fd = CAST_DOWN_EXPLICIT(int, uap->sem);
        struct fileproc *fp;
        struct pseminfo * pinfo;
-       struct psemnode * pnode ;
+       struct psemnode * pnode;
        kern_return_t kret;
        int error;
 
        error = fp_getfpsem(p, fd, &fp, &pnode);
-       if (error)
-               return (error);
-       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL )  {
+       if (error) {
+               return error;
+       }
+       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL) {
                error = EINVAL;
                goto out;
        }
@@ -857,8 +873,8 @@ sem_wait_nocancel(proc_t p, struct sem_wait_nocancel_args *uap, __unused int32_t
                error = EINVAL;
                goto out;
        }
-       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED)) 
-                       != PSEM_ALLOCATED) {
+       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED))
+           != PSEM_ALLOCATED) {
                PSEM_SUBSYS_UNLOCK();
                error = EINVAL;
                goto out;
@@ -890,25 +906,25 @@ sem_wait_nocancel(proc_t p, struct sem_wait_nocancel_args *uap, __unused int32_t
        }
 out:
        fp_drop(p, fd, fp, 0);
-       return(error);
-
+       return error;
 }
 
 int
 sem_trywait(proc_t p, struct sem_trywait_args *uap, __unused int32_t *retval)
 {
-       int fd = CAST_DOWN_EXPLICIT(int,uap->sem);
+       int fd = CAST_DOWN_EXPLICIT(int, uap->sem);
        struct fileproc *fp;
        struct pseminfo * pinfo;
-       struct psemnode * pnode ;
+       struct psemnode * pnode;
        kern_return_t kret;
        mach_timespec_t wait_time;
        int error;
-       
+
        error = fp_getfpsem(p, fd, &fp, &pnode);
-       if (error)
-               return (error);
-       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL )  {
+       if (error) {
+               return error;
+       }
+       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL) {
                error = EINVAL;
                goto out;
        }
@@ -918,8 +934,8 @@ sem_trywait(proc_t p, struct sem_trywait_args *uap, __unused int32_t *retval)
                error = EINVAL;
                goto out;
        }
-       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED)) 
-                       != PSEM_ALLOCATED) {
+       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED))
+           != PSEM_ALLOCATED) {
                PSEM_SUBSYS_UNLOCK();
                error = EINVAL;
                goto out;
@@ -956,23 +972,24 @@ sem_trywait(proc_t p, struct sem_trywait_args *uap, __unused int32_t *retval)
        }
 out:
        fp_drop(p, fd, fp, 0);
-       return(error);
+       return error;
 }
 
 int
 sem_post(proc_t p, struct sem_post_args *uap, __unused int32_t *retval)
 {
-       int fd = CAST_DOWN_EXPLICIT(int,uap->sem);
+       int fd = CAST_DOWN_EXPLICIT(int, uap->sem);
        struct fileproc *fp;
        struct pseminfo * pinfo;
-       struct psemnode * pnode ;
+       struct psemnode * pnode;
        kern_return_t kret;
        int error;
 
        error = fp_getfpsem(p, fd, &fp, &pnode);
-       if (error)
-               return (error);
-       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL )  {
+       if (error) {
+               return error;
+       }
+       if (((pnode = (struct psemnode *)fp->f_data)) == PSEMNODE_NULL) {
                error = EINVAL;
                goto out;
        }
@@ -982,8 +999,8 @@ sem_post(proc_t p, struct sem_post_args *uap, __unused int32_t *retval)
                error = EINVAL;
                goto out;
        }
-       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED)) 
-                       != PSEM_ALLOCATED) {
+       if ((pinfo->psem_flags & (PSEM_DEFINED | PSEM_ALLOCATED))
+           != PSEM_ALLOCATED) {
                PSEM_SUBSYS_UNLOCK();
                error = EINVAL;
                goto out;
@@ -1015,43 +1032,43 @@ sem_post(proc_t p, struct sem_post_args *uap, __unused int32_t *retval)
        }
 out:
        fp_drop(p, fd, fp, 0);
-       return(error);
+       return error;
 }
 
 static int
 psem_close(struct psemnode *pnode, __unused int flags)
 {
-       int error=0;
+       int error = 0;
        struct pseminfo *pinfo;
 
        PSEM_SUBSYS_LOCK();
-       if ((pinfo = pnode->pinfo) == PSEMINFO_NULL){
+       if ((pinfo = pnode->pinfo) == PSEMINFO_NULL) {
                PSEM_SUBSYS_UNLOCK();
-               return(EINVAL);
+               return EINVAL;
        }
 
        if ((pinfo->psem_flags & PSEM_ALLOCATED) != PSEM_ALLOCATED) {
                PSEM_SUBSYS_UNLOCK();
-               return(EINVAL);
+               return EINVAL;
        }
 #if DIAGNOSTIC
-       if(!pinfo->psem_usecount) {
+       if (!pinfo->psem_usecount) {
                kprintf("negative usecount in psem_close\n");
        }
 #endif /* DIAGNOSTIC */
        pinfo->psem_usecount--;
 
-       if ((pinfo->psem_flags & PSEM_REMOVED) && !pinfo->psem_usecount) {
+       if ((pinfo->psem_flags & PSEM_REMOVED) && !pinfo->psem_usecount) {
                PSEM_SUBSYS_UNLOCK();
                /* lock dropped as only semaphore is destroyed here */
                error = psem_delete(pinfo);
-               FREE(pinfo,M_SHM);
+               FREE(pinfo, M_SHM);
        } else {
                PSEM_SUBSYS_UNLOCK();
        }
        /* subsystem lock is dropped when we get here */
        FREE(pnode, M_SHM);
-       return (error);
+       return error;
 }
 
 static int
@@ -1065,10 +1082,10 @@ psem_closefile(struct fileglob *fg, __unused vfs_context_t ctx)
         */
        error =  psem_close(((struct psemnode *)fg->fg_data), fg->fg_flag);
 
-       return(error);
+       return error;
 }
 
-static int 
+static int
 psem_delete(struct pseminfo * pinfo)
 {
        kern_return_t kret;
@@ -1081,54 +1098,17 @@ psem_delete(struct pseminfo * pinfo)
        switch (kret) {
        case KERN_INVALID_ADDRESS:
        case KERN_PROTECTION_FAILURE:
-               return (EINVAL);
+               return EINVAL;
        case KERN_ABORTED:
        case KERN_OPERATION_TIMED_OUT:
-               return (EINTR);
+               return EINTR;
        case KERN_SUCCESS:
-               return(0);
+               return 0;
        default:
-               return (EINVAL);
+               return EINVAL;
        }
 }
 
-static int
-psem_read(__unused struct fileproc *fp, __unused struct uio *uio, 
-                 __unused int flags, __unused vfs_context_t ctx)
-{
-       return(ENOTSUP);
-}
-
-static int
-psem_write(__unused struct fileproc *fp, __unused struct uio *uio, 
-                  __unused int flags, __unused vfs_context_t ctx)
-{
-       return(ENOTSUP);
-}
-
-static int
-psem_ioctl(__unused struct fileproc *fp, __unused u_long com, 
-                       __unused caddr_t data, __unused vfs_context_t ctx)
-{
-       return(ENOTSUP);
-}
-
-static int
-psem_select(__unused struct fileproc *fp, __unused int which, 
-                       __unused void *wql, __unused vfs_context_t ctx)
-{
-       return(ENOTSUP);
-}
-
-static int
-psem_kqfilter(__unused struct fileproc *fp, struct knote *kn, 
-                               __unused vfs_context_t ctx)
-{
-       kn->kn_flags = EV_ERROR;
-       kn->kn_data = ENOTSUP;
-       return 0;
-}
-
 int
 fill_pseminfo(struct psemnode *pnode, struct psem_info * info)
 {
@@ -1136,29 +1116,29 @@ fill_pseminfo(struct psemnode *pnode, struct psem_info * info)
        struct vinfo_stat  *sb;
 
        PSEM_SUBSYS_LOCK();
-       if ((pinfo = pnode->pinfo) == PSEMINFO_NULL){
+       if ((pinfo = pnode->pinfo) == PSEMINFO_NULL) {
                PSEM_SUBSYS_UNLOCK();
-               return(EINVAL);
+               return EINVAL;
        }
 
 #if 0
        if ((pinfo->psem_flags & PSEM_ALLOCATED) != PSEM_ALLOCATED) {
                PSEM_SUBSYS_UNLOCK();
-               return(EINVAL);
+               return EINVAL;
        }
 #endif
 
        sb = &info->psem_stat;
        bzero(sb, sizeof(struct vinfo_stat));
 
-       sb->vst_mode = pinfo->psem_mode;
-       sb->vst_uid = pinfo->psem_uid;
-       sb->vst_gid = pinfo->psem_gid;
-       sb->vst_size = pinfo->psem_usecount;
-       bcopy(&pinfo->psem_name[0], &info->psem_name[0], PSEMNAMLEN+1);
+       sb->vst_mode = pinfo->psem_mode;
+       sb->vst_uid = pinfo->psem_uid;
+       sb->vst_gid = pinfo->psem_gid;
+       sb->vst_size = pinfo->psem_usecount;
+       bcopy(&pinfo->psem_name[0], &info->psem_name[0], PSEMNAMLEN + 1);
 
        PSEM_SUBSYS_UNLOCK();
-       return(0);
+       return 0;
 }
 
 #if CONFIG_MACF
@@ -1172,12 +1152,12 @@ psem_label_associate(struct fileproc *fp, struct vnode *vp, vfs_context_t ctx)
        pnode = (struct psemnode *)fp->f_fglob->fg_data;
        if (pnode != NULL) {
                psem = pnode->pinfo;
-               if (psem != NULL)
+               if (psem != NULL) {
                        mac_posixsem_vnode_label_associate(
                                vfs_context_ucred(ctx), psem, psem->psem_label,
                                vp, vp->v_label);
+               }
        }
        PSEM_SUBSYS_UNLOCK();
 }
 #endif
-