fixed success check