deepblue/ipmitool
1
0
Fork 0
mirror of https://github.com/ipmitool/ipmitool.git synced 2025-07-05 20:23:35 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ID: 3600928 - 'lib/ipmi_pef.c' use 'else if ()' instead of 'if ()'

Browse Source 
Commit trades 'if ()' for 'else if ()' and removes one pointless 'if ()' since
rsp isn't NULL at this point.
This commit is contained in:
Zdenek Styblik
2013-02-13 11:35:21 +00:00
parent c726a09482
commit ed1865da0a
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
ipmitool/lib/ipmi_pef.c
View File

@ -193,18 +193,17 @@ ipmi_pef_msg_exchange(struct ipmi_intf * intf, struct ipmi_rq * req, char * txt)
// common IPMItool rqst/resp handling // common IPMItool rqst/resp handling
*/ */
struct ipmi_rs * rsp = intf->sendrecv(intf, req); struct ipmi_rs * rsp = intf->sendrecv(intf, req);
if (!rsp) if (!rsp) {
return(NULL); return(NULL);
if (rsp->ccode == 0x80) { } else if (rsp->ccode == 0x80) {
return(NULL); /* Do not output error, just unsupported parameters */ return(NULL); /* Do not output error, just unsupported parameters */
} } else if (rsp->ccode) {
else if (rsp->ccode) { lprintf(LOG_ERR, " **Error %x in '%s' command", rsp->ccode, txt);
lprintf(LOG_ERR, " **Error %x in '%s' command",
rsp ? rsp->ccode : 0, txt);
return(NULL); return(NULL);
} }
if (verbose > 2) if (verbose > 2) {
printbuf(rsp->data, rsp->data_len, txt); printbuf(rsp->data, rsp->data_len, txt);
}
return(rsp); return(rsp);
} }