IntDestroyCurIconObject and IntDestroyMenuObject corrupt the cursor/menu list when called multiple times

The destruction shouldn't happen multiple times in the first place. However currently UserDeleteObject will fail if there are outstanding references on the object, so these functions can be called multiple times.

Currently they both unconditionally call RemoveEntryList on their ListEntry. Repeating this can cause list corruption though.