fix a bug in inorder successor in a BST