modified nacl_sync.h to have NACL_WUR for all functions that return a

src/trusted/service_runtime/nacl_syscall_common.c

 /*
  * Copyright (c) 2011 The Native Client Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 /*
  * NaCl service run-time, non-platform specific system call helper routines.
  */
 #include <sys/types.h>
@@ skipping 272 matching lines @@
               "npages 0x%"NACL_PRIxS"\n",
               ent->page_num, ent->npages);
       nap->break_addr = new_break;
       break_addr = new_break;
     }
   }
 
 
 
 cleanup:
-  (void) NaClMutexUnlock(&nap->mu);
+  NaClXMutexUnlock(&nap->mu);
 cleanup_no_lock:
   NaClSysCommonThreadSyscallLeave(natp);
 
   /*
    * This cast is safe because the incoming value (new_break) cannot
    * exceed the user address space--even though its type (uintptr_t)
    * theoretically allows larger values.
    */
   rv = (int32_t) break_addr;
 
@@ skipping 87 matching lines @@
 
   NaClLog(1, "Exit syscall handler: %d\n", status);
   NaClSysCommonThreadSyscallEnter(natp);
 
   nap = natp->nap;
   NaClSyncQueueQuit(&nap->work_queue);
 
   NaClXMutexLock(&nap->mu);
   nap->exit_status = status;
   nap->running = 0;
-  NaClCondVarSignal(&nap->cv);
+  NaClXCondVarSignal(&nap->cv);
   NaClXMutexUnlock(&nap->mu);
 
   NaClSysCommonThreadSuicide(natp);
   /* NOTREACHED */
   return -NACL_ABI_EINVAL;
 }
 
 int32_t NaClCommonSysThreadExit(struct NaClAppThread  *natp,
                                 int32_t               *stack_flag) {
   uintptr_t sys_stack_flag;
@@ skipping 72 matching lines @@
     if (NULL == desc_obj_ptr) {
       retval = -NACL_ABI_EBADF;
       goto done;
     }
     if (NACL_DESC_CONN_CAP != NACL_VTBL(NaClDesc, desc_obj_ptr)->typeTag &&
         NACL_DESC_CONN_CAP_FD != NACL_VTBL(NaClDesc, desc_obj_ptr)->typeTag) {
       retval = -NACL_ABI_EINVAL;
       goto done;
     }
     /* write */
-    NaClMutexLock(&natp->nap->mu);
+    NaClXMutexLock(&natp->nap->mu);
     NaClDescUnref(natp->nap->name_service_conn_cap);
     natp->nap->name_service_conn_cap = desc_obj_ptr;
-    NaClMutexUnlock(&natp->nap->mu);
+    NaClXMutexUnlock(&natp->nap->mu);
     retval = 0;
   }
 
  done:
   NaClSysCommonThreadSyscallLeave(natp);
   return retval;
 }
 
 int32_t NaClCommonSysDup(struct NaClAppThread *natp,
                          int                  oldfd) {
@@ skipping 2218 matching lines @@
     retval = -NACL_ABI_EBADF;
     goto cleanup;
   }
 
   retval = (*((struct NaClDescVtbl const *) desc->base.vtbl)->GetValue)(desc);
   NaClDescUnref(desc);
 cleanup:
   NaClSysCommonThreadSyscallLeave(natp);
   return retval;
 }