[PATCH phosphor-host-ipmid] Fix two issues in Issue #80

[Patrick Williams]

On Mon, Jun 27, 2016 at 08:40:42AM -0500, OpenBMC Patches wrote:
> From: tomjose <tomjoseph at in.ibm.com>
> 
> Corrected the comment for the Get Channel Info command to
> reflect the logic in the code. For issue two, corrected the
> condition for returning the unspecifed error.
> 
> Resolves: #80
> See also: #77,#78

These should be of the form 'openbmc/host-ipmid#80'.

https://help.github.com/articles/closing-issues-via-commit-messages/

Also would prefer not to have "Tag: Info" because sometimes git and/or
tools ascribe meaning to them.  This is the "Footer" of the message and
is the same place that 'Reviewed-By' tags and Gerrit 'Change-Id' tags
go.  Best to not use something in that format to avoid confusing a tool.

> ---
>  apphandler.C | 4 +++-
>  ipmid.C      | 2 +-
>  2 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/apphandler.C b/apphandler.C
> index 71dba66..9ce8829 100644
> --- a/apphandler.C
> +++ b/apphandler.C
> @@ -443,7 +443,9 @@ ipmi_ret_t ipmi_app_channel_info(ipmi_netfn_t netfn, ipmi_cmd_t cmd,
>  
>      printf("IPMI APP GET CHANNEL INFO\n");
>  
> -    // I"m only supporting channel 1.  0xE is the 'default channel'
> +    // The supported channels numbers are 1 and 8.
> +    // Channel Number E is used as way to identify the current channel
> +    // that the command is being is received from.
>      if (*p == 0xe || *p == 1 || *p == 8) {
>  
>          *data_len = sizeof(resp);
> diff --git a/ipmid.C b/ipmid.C
> index 7354958..374ede2 100644
> --- a/ipmid.C
> +++ b/ipmid.C
> @@ -273,7 +273,7 @@ static int handle_ipmi_command(sd_bus_message *m, void *user_data, sd_bus_error
>      {
>          fprintf(stderr,"ERROR:[0x%X] handling NetFn:[0x%X], Cmd:[0x%X]\n",r, netfn, cmd);
>  
> -        if(r == -1) {
> +        if(r < 0) {
>             response[0] = IPMI_CC_UNSPECIFIED_ERROR;
>          }
>      }

Code changes look good.

-- 
Patrick Williams
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20160627/90712b6c/attachment-0001.sig>