prevent dead-lock by removing mutex lock in errorHandlerProc

- setting int should be OK even without lock as there is no synchronization in reader side
- dead-lock can happen inside constructor if the same error handler is already
  set and if error happens inside constructor in place like XSync

Change-Id: I2f401067e0555ae092df23f57cc9ab054a1555d7