On 08/20/2017 11:46 PM, miny...@acm.org wrote: > From: Corey Minyard <cminy...@mvista.com> > > The minimum message size was on the wrong commands, for getting > the time it's zero and for setting the time it's 6. > > Signed-off-by: Corey Minyard <cminy...@mvista.com>
Reviewed-by: Cédric Le Goater <c...@kaod.org> > --- > hw/ipmi/ipmi_bmc_sim.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c > index 277c28c..cc068f2 100644 > --- a/hw/ipmi/ipmi_bmc_sim.c > +++ b/hw/ipmi/ipmi_bmc_sim.c > @@ -1802,8 +1802,8 @@ static const IPMICmdHandler storage_cmds[] = { > [IPMI_CMD_GET_SEL_ENTRY] = { get_sel_entry, 8 }, > [IPMI_CMD_ADD_SEL_ENTRY] = { add_sel_entry, 18 }, > [IPMI_CMD_CLEAR_SEL] = { clear_sel, 8 }, > - [IPMI_CMD_GET_SEL_TIME] = { get_sel_time, 6 }, > - [IPMI_CMD_SET_SEL_TIME] = { set_sel_time }, > + [IPMI_CMD_GET_SEL_TIME] = { get_sel_time }, > + [IPMI_CMD_SET_SEL_TIME] = { set_sel_time, 6 }, > }; > > static const IPMINetfn storage_netfn = { >