deepblue/ipmitool
1
0
Fork 0
mirror of https://github.com/ipmitool/ipmitool.git synced 2025-05-10 18:47:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ID: 3600910 - 'lib/ipmi_delloem.c' use 'else if' instead of 'if', use 'rc'

Browse Source 
Commit changes are related to ipmi_ldc_set_lock() in 'lib/ipmi_delloem.c':
* use 'else if' instead of 'if' since it's the same given the context and
probably better
* use 'rc' variable, since it's there, rather than 'return'

Note: ipmi_lcd_set_lock() would use some re-write.

Reported-by: Ales Ledvinka
This commit is contained in:
Zdenek Styblik 2013-02-12 12:23:11 +00:00
parent c53a3cbb8e
commit a70de993cf
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ipmitool/lib/ipmi_delloem.c
View File

@ -1463,10 +1463,10 @@ ipmi_lcd_set_lock(struct ipmi_intf * intf, char lock)
lprintf(LOG_ERR, " Error setting LCD status"); lprintf(LOG_ERR, " Error setting LCD status");
rc= -1; rc= -1;
} }
if ((rsp->ccode == 0xc1)||(rsp->ccode == 0xcb)) else if ((rsp->ccode == 0xc1) || (rsp->ccode == 0xcb))
{ {
lprintf(LOG_ERR, " Error getting LCD status: Command not supported on this system."); lprintf(LOG_ERR, " Error getting LCD status: Command not supported on this system.");
return -1; rc = -1;
} }
else if (rsp->ccode > 0) else if (rsp->ccode > 0)
{ {