YUCK!
First off, you should not hack up a UDF and turn it into quasi-inline code.
Second, lose the GOTOs.
Third, how would you call the UDF and pass parms to it?
In your fist post, you said you needed to know what OU a computer is in but it looks to me you are turning it around to see if a computer is in a particular OU. That is not the same thing. What is wrong with just using InContainer() without having to modify anything?
_________________________
Give a man a fish and he will be back for more. Slap him with a fish and he will go away forever.
