On Fri, Jun 01, 2018 at 08:50:16AM -0700, dsah...@kernel.org wrote:
> From: David Ahern <dsah...@gmail.com>
> 
> A recent commit changed rtnl_talk_* to return the response message in
> allocated memory so callers need to free it. The change to name_is_vrf
> did not save the device index which is pointing to a struct inside the
> now allocated and freed memory resulting in garbage getting returned
> in some cases.
> 
> Fix by using a stack variable to save the return value and only set
> it to ifi->ifi_index after all checks are done and before the answer
> buffer is freed.
> 
> Fixes: 86bf43c7c2fdc ("lib/libnetlink: update rtnl_talk to support malloc 
> buff at run time")
> Cc: Hangbin Liu <liuhang...@gmail.com>
> Cc: Phil Sutter <p...@nwl.cc>
> Signed-off-by: David Ahern <dsah...@gmail.com>

Acked-by: Phil Sutter <p...@nwl.cc>

Thanks, Phil

Reply via email to