From fb6e311d272fcdcf44cf795c9c62008c28eb0fc4 Mon Sep 17 00:00:00 2001 From: Zdenek Styblik Date: Sat, 25 Jul 2015 12:53:16 +0200 Subject: [PATCH] ID:367 - Fix building of non-bridged LAN interface commands When bridging is used (-t specifies an address which is different from 20h), presession command must go to BMC_SLAVE_ADDR, not to target_addr. BMC_SLAVE_ADDR is always used as the responder address in outer Send Message request, when message is forwarded. Use the same approach for non-bridged commands. Commit for: Dmitry Bazhenov --- src/plugins/lan/lan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/lan/lan.c b/src/plugins/lan/lan.c index eed4a82..4169428 100644 --- a/src/plugins/lan/lan.c +++ b/src/plugins/lan/lan.c @@ -818,7 +818,7 @@ ipmi_lan_build_cmd(struct ipmi_intf * intf, struct ipmi_rq * req, int isRetry) } /* ipmi message header */ - msg[len++] = intf->target_addr; + msg[len++] = entry->bridging_level ? intf->target_addr : IPMI_BMC_SLAVE_ADDR; msg[len++] = req->msg.netfn << 2 | (req->msg.lun & 3); tmp = len - cs; msg[len++] = ipmi_csum(msg+cs, tmp);