tree b27ece0f00a0ee072ef66d178ee7879b0cc73756
parent 0f1b2fac03361c5d2bac34e4b19922c60c5c06c6
author Xin Long <lucien.xin@gmail.com> 1554106524 +0800
committer Jiri Pirko <jiri@mellanox.com> 1555486692 +0200

teamd: tdport has to exist if item->per_port is set in __find_by_item_path

The issue can be reproduced by:

  # teamd -c '{"device":"team0", "runner":{"name":"lacp"}}' &
  # teamdctl team0 state item set runner.aggregator.selected true

teamd process will abort in lacp_port_state_aggregator_selected_set()
as gsc->info.tdport was set to NULL in teamd_state_item_value_set()

The item 'runner.aggregator.selected' is of per_port = true, and it
shouldn't allow to call its setter/getter().

This patch is to add the check for it in __find_by_item_path() called
by teamd_state_item_value_get/set().

Fixes: 6c00aaf02553 ("teamd: add support for state item write operation")
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>