service: Don't iterate over changing sequence

service: Don't iterate over changing sequence

__connman_service_disconnect_all() should not iterate over
the service_list directly because the ordering of the
services can change when disconnecting a service (e.g. default
service). The iterator might be invalid after a
__connman_service_disconnect() call.

It's better to iterate over a temporary list.