should not we exit 1 if success==false?