commit f75b87815d28cc5368d00a0c2d612158399822cc
author Patrick Williams <patrick@stwcx.xyz> Thu Jun 30 16:19:59 2016 -0500
committer GitHub <noreply@github.com> Thu Jun 30 16:19:59 2016 -0500
tree 299a822734c67d4bca7b7d42d4fd4d86610048be
parent b7bcda57ee39616e8937194d281e2476e6ea8df2
parent 7ec0add13801ea07145adf703ad35af10f492a24

Merge pull request #95 from tomjoseph83/issue80

Fix two issues in Issue #80

apphandler.C

diff --git a/apphandler.C b/apphandler.C
index d014f5b..b8cc0aa 100644
--- a/apphandler.C
+++ b/apphandler.C
@@ -445,7 +445,9 @@
 
     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);

ipmid.C

diff --git a/ipmid.C b/ipmid.C
index 7354958..374ede2 100644
--- a/ipmid.C
+++ b/ipmid.C
@@ -273,7 +273,7 @@
     {
         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;
         }
     }