transport: unify return values and exit point from transport_push()

transport: unify return values and exit point from transport_push()

It seems there is no reason to return 1 instead of -1 when push_refs()
is not set in transport vtable. Let's unify the error return values and
use the done label as a single exit point from transport_push().

Suggested-by: Ævar Arnfjörð Bjarmason <redacted>
Signed-off-by: Frantisek Hrbata <redacted>
Reviewed-by: Josh Steadmon <redacted>
Signed-off-by: Junio C Hamano <redacted>