ipmitool delloem: ipmitool delloem extension always return success - fix it

The delloem extension functions all return sane error codes, but they
are not used and not passed on. Fix it.

Signed-off-by: Thomas Renninger <trenn@suse.de>
This commit is contained in:
Paul Zirnik 2014-07-08 18:25:53 +02:00 committed by Thomas Renninger
parent d79b0e05af
commit 2f57897501

View File

@ -263,22 +263,22 @@ ipmi_delloem_main(struct ipmi_intf * intf, int argc, char ** argv)
return 0; return 0;
} }
if (0 ==strncmp(argv[current_arg], "lcd\0", 4)) { if (0 ==strncmp(argv[current_arg], "lcd\0", 4)) {
ipmi_delloem_lcd_main(intf,argc,argv); rc = ipmi_delloem_lcd_main(intf,argc,argv);
} else if (strncmp(argv[current_arg], "mac\0", 4) == 0) { } else if (strncmp(argv[current_arg], "mac\0", 4) == 0) {
/* mac address*/ /* mac address*/
ipmi_delloem_mac_main(intf,argc,argv); rc = ipmi_delloem_mac_main(intf,argc,argv);
} else if (strncmp(argv[current_arg], "lan\0", 4) == 0) { } else if (strncmp(argv[current_arg], "lan\0", 4) == 0) {
/* lan address*/ /* lan address*/
ipmi_delloem_lan_main(intf,argc,argv); rc = ipmi_delloem_lan_main(intf,argc,argv);
} else if (strncmp(argv[current_arg], "setled\0", 7) == 0) { } else if (strncmp(argv[current_arg], "setled\0", 7) == 0) {
/* SetLED support */ /* SetLED support */
ipmi_delloem_setled_main(intf,argc,argv); rc = ipmi_delloem_setled_main(intf,argc,argv);
} else if (strncmp(argv[current_arg], "powermonitor\0", 13) == 0) { } else if (strncmp(argv[current_arg], "powermonitor\0", 13) == 0) {
/*Powermanagement report processing*/ /*Powermanagement report processing*/
ipmi_delloem_powermonitor_main(intf,argc,argv); rc = ipmi_delloem_powermonitor_main(intf,argc,argv);
} else if (strncmp(argv[current_arg], "vFlash\0", 7) == 0) { } else if (strncmp(argv[current_arg], "vFlash\0", 7) == 0) {
/* vFlash Support */ /* vFlash Support */
ipmi_delloem_vFlash_main(intf,argc,argv); rc = ipmi_delloem_vFlash_main(intf,argc,argv);
} else { } else {
usage(); usage();
return -1; return -1;