Bugzilla bug 226356: test for NULL before dereferencing a pointer. The

patch is contributed by timeless@bemail.org.  r=wtc.
Modified Files: bthreads/btmon.c pthreads/ptsynch.c threads/prmon.c

nsprpub/pr/src/bthreads/btmon.c

@@ -78,7 +78,10 @@ PR_IMPLEMENT(PRMonitor*)
 PR_IMPLEMENT(PRMonitor*) PR_NewNamedMonitor(const char* name)
 {
     PRMonitor* mon = PR_NewMonitor();
-    mon->name = name;
+    if( mon )
+    {
+        mon->name = name;
+    }
     return mon;
 }
 

nsprpub/pr/src/pthreads/ptsynch.c

@@ -471,7 +471,8 @@ PR_IMPLEMENT(PRMonitor*) PR_NewMonitor(void)
 PR_IMPLEMENT(PRMonitor*) PR_NewNamedMonitor(const char* name)
 {
     PRMonitor* mon = PR_NewMonitor();
-    mon->name = name;
+    if (mon)
+        mon->name = name;
     return mon;
 }
 

nsprpub/pr/src/threads/prmon.c

@@ -68,7 +68,8 @@ PR_IMPLEMENT(PRMonitor*) PR_NewMonitor()
 PR_IMPLEMENT(PRMonitor*) PR_NewNamedMonitor(const char* name)
 {
     PRMonitor* mon = PR_NewMonitor();
-    mon->name = name;
+    if (mon)
+        mon->name = name;
     return mon;
 }