odrefresh should exit non-zero for invalid argument

This fixes two cases.

1. "Expected 1 argument, but have %d."
2. unknown action

Because of 1, the loop is actually not necessary (as pointed out by the
compiler), so removed.

With 2 now exits, the default return is never eached, thus deleted.

Bug: 205750213
Test: atest art_standalone_odrefresh_tests
Test: atest ComposHostTestCases
Change-Id: Ifc198e888045e633a63a918509faf9a0fe1e24fb