Yes; locking structures are /generally/ re-entrant, meaning if you already
hold the lock you can effectively take out another... to avoid scenarios
with deadlocking yourself.

Marc

DaTurk,

Yep, that's fine because it is locking on the same object.

What you do need to look out for is the order of locks when different
objects are used.  A common deadlock scenario exists when you have two
different critical sections that lock on two different objects in
different orders.  There are plenty of others, but this is common
enough to mention.  For example, the following code would eventually
deadlock.

void CritialSection1()
{
 lock (a)
 {
   lock (b)
   {
     // whatever
   }
 }
}

void CritialSection2()
{
 lock (b)
 {
   lock (a)
   {
     // whatever
   }
 }
}

Brian

While that works fine, I would be concerned with the fact that you
noticed the double-lock after the fact.  If you're not fully aware of
what is being locked when and what is already locked when certain code
is running, then it's more likely you'll create deadlock situations or
innefficient code (i.e., not a deadlock but code that's unnecessarily
waiting on other code).

In general it's a good practice to keep locked sections as small as
possible and be very aware of what is running inside locked sections.

Sam

------------------------------------------------------------
We're hiring!  B-Line Medical is seeking Mid/Sr. .NET
Developers for exciting positions in medical product
development in MD/DC.  Work with a variety of technologies
in a relaxed team environment.  See ads on Dice.com.