Get rid of return 1; all together. Don't replace it with break, you will have the same problem. Also, move ShowPlayerDialog outside both loops.
No he won't there is two loops here, once he finds the driver he can break out of that loop then continue checking the other cars if he doesn't then it's going to check the rest of players uselessly read the code.