That snippet makes no sense.

• <> returns undef on error, so checking for an error in the loop is nonsense.
• A true $! doesn't mean an error occured.
• A false $! doesn't mean no error occured.