Re: [PATCH 0/6] MR3572: mf: Make session_get_node_object() more robust.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/session.c:
> -
>
> struct topo_node *node = NULL, *found = NULL;
>
> LIST_FOR_EACH_ENTRY(node, &session->presentation.nodes, struct topo_node, entry)
> {
>     if (node->type == node_type && object == node->object.object)
> -
>    {
> -
>        found = node;
>        break;
> -
>    }
>
> }
> - return node;
> - if (!found)
> -
>    WARN("Failed to find object %p.\n", object);
> - return found;
>
> }

This is too complicated, simply return "node" from within the loop, and NULL otherwise.