Re: [ros-dev] Alex has fixed the laptop problem

[Thomas Weidenmueller]

James Tabor wrote:

>   This test same as above using the taskmgr.exe while
> compiling Ros on Ros.
>  The branch is Head or current SVN,
>
>  http://rafb.net/paste/results/qOCyyY29.html 
I believe this is due to a bug in the object parsing code where it 
doesn't reference an object when walking into deeper levels. Could you 
please try my attached patch on SVN trunk if it fixes it?

Best Regards,
Thomas

Index: cm/regobj.c
===================================================================
--- cm/regobj.c	(revision 13818)
+++ cm/regobj.c	(working copy)
@@ -212,11 +212,6 @@
 	      return(STATUS_REPARSE);
 	    }
 	}
-
-      ObReferenceObjectByPointer(FoundObject,
-				 STANDARD_RIGHTS_REQUIRED,
-				 NULL,
-				 UserMode);
     }
 
   DPRINT("CmiObjectParse: %s\n", FoundObject->Name);
Index: ob/object.c
===================================================================
--- ob/object.c	(revision 13818)
+++ ob/object.c	(working copy)
@@ -439,14 +439,16 @@
 	     /* reparse the object path */
 	     NextObject = NameSpaceRoot;
 	     current = PathString.Buffer;
-	     
+	  }
+
+	if (NextObject != NULL)
+	  {
 	     ObReferenceObjectByPointer(NextObject,
 					DIRECTORY_TRAVERSE,
 					NULL,
 					UserMode);
 	  }
-
-	if (NextObject == NULL)
+        else
 	  {
 	     break;
 	  }