Don't unnecessarily call XDefineCursor(). Call it only when freeing the cursor resource, and call it before freeing the cursor resource then to avoid problems with a defined cursor which resource has already been freed. Additionally, use symbolic constant instead of numeric value.