Hi,

On Thu, 2012-04-12 at 11:33 +0200, Daniel Wagner wrote:
> +static connman_bool_t is_autoconnect(struct connman_network *network,
> +                                     enum connman_service_type type)

Instead of refactoring code into a new function, I'd just do this:

-       for (i = 0; auto_connect_types[i] != 0; i += 1) {
+       for (i = 0; auto_connect_types != NULL &&
+                    auto_connect_types[i] != 0; i++) {

Thanks for the bug hunting!

Cheers,

        Patrik


_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to