diff options
author | roopa <roopa@cumulusnetworks.com> | 2013-02-15 10:26:30 -0800 |
---|---|---|
committer | Thomas Graf <tgraf@suug.ch> | 2013-02-24 17:11:36 +0100 |
commit | ded20487fd631f7fcfc8f3cc547f6a8852501b83 (patch) | |
tree | 3e63b437994799191c252298dda51c872f4387cf /src | |
parent | 8f151fadda62bdbd363b8d4a0167d5d29bb5163b (diff) | |
download | libnl-ded20487fd631f7fcfc8f3cc547f6a8852501b83.tar.gz |
link: Fix rtnl_link_af_data_compare return value
This patch fixes a bug where because of the af_ops check
being first in the function, we were returning ~0 if af_ops
was null even if both objects really did not have af_data
and we should be returning 0.
Its better to have the af_data present check before anything else.
So, Rearranged some of the code in rtnl_link_af_data_compare.
Changes include:
- Do the attribute present check before anything else
- If ao_compare op not present, return ~0
Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
Reviewed-by: Nolan Leake <nolan@cumulusnetworks.com>
Reviewed-by: Shrijeet Mukherjee <shm@cumulusnetworks.com>
Reviewed-by: Wilson Kok <wkok@cumulusnetworks.com>
Signed-off-by: Thomas Graf <tgraf@suug.ch>
Diffstat (limited to 'src')
0 files changed, 0 insertions, 0 deletions