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: 153 - 'lib/ipmi_firewall.c' - printf() used instead of lprintf()

Browse Source 
Use lprintf() for error/help messages.
This commit is contained in:
Zdenek Styblik 2013-07-09 04:20:17 +00:00
parent c048c9f311
commit 0ab78dbf02
1 changed files with 23 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
ipmitool/lib/ipmi_firewall.c
View File

@ -47,14 +47,22 @@
static void static void
printf_firewall_usage(void) printf_firewall_usage(void)
{ {
printf("Firmware Firewall Commands:\n"); lprintf(LOG_NOTICE,
printf("\tinfo [channel H] [lun L]\n"); "Firmware Firewall Commands:");
printf("\tinfo [channel H] [lun L [netfn N [command C [subfn S]]]]\n"); lprintf(LOG_NOTICE,
printf("\tenable [channel H] [lun L [netfn N [command C [subfn S]]]]\n"); "\tinfo [channel H] [lun L]");
printf("\tdisable [channel H] [lun L [netfn N [command C [subfn S]]]] [force])\n"); lprintf(LOG_NOTICE,
printf("\treset [channel H] \n"); "\tinfo [channel H] [lun L [netfn N [command C [subfn S]]]]");
printf("\t\twhere H is a Channel, L is a LUN, N is a NetFn,\n"); lprintf(LOG_NOTICE,
printf("\t\tC is a Command and S is a Sub-Function\n"); "\tenable [channel H] [lun L [netfn N [command C [subfn S]]]]");
lprintf(LOG_NOTICE,
"\tdisable [channel H] [lun L [netfn N [command C [subfn S]]]] [force])");
lprintf(LOG_NOTICE,
"\treset [channel H]");
lprintf(LOG_NOTICE,
"\t\twhere H is a Channel, L is a LUN, N is a NetFn,");
lprintf(LOG_NOTICE,
"\t\tC is a Command and S is a Sub-Function");
} }
// print n bytes of bit field bf (if invert, print ~bf) // print n bytes of bit field bf (if invert, print ~bf)
@ -134,31 +142,31 @@ ipmi_firewall_parse_args(int argc, char ** argv, struct ipmi_function_params * p
return (-1); return (-1);
} }
if (p->subfn >= MAX_SUBFN) { if (p->subfn >= MAX_SUBFN) {
printf("subfn is out of range (0-%d)\n", MAX_SUBFN-1); lprintf(LOG_ERR, "subfn is out of range (0-%d)", MAX_SUBFN-1);
return -1; return -1;
} }
if (p->command >= MAX_COMMAND) { if (p->command >= MAX_COMMAND) {
printf("command is out of range (0-%d)\n", MAX_COMMAND-1); lprintf(LOG_ERR, "command is out of range (0-%d)", MAX_COMMAND-1);
return -1; return -1;
} }
if (p->netfn >= MAX_NETFN) { if (p->netfn >= MAX_NETFN) {
printf("netfn is out of range (0-%d)\n", MAX_NETFN-1); lprintf(LOG_ERR, "netfn is out of range (0-%d)", MAX_NETFN-1);
return -1; return -1;
} }
if (p->lun >= MAX_LUN) { if (p->lun >= MAX_LUN) {
printf("lun is out of range (0-%d)\n", MAX_LUN-1); lprintf(LOG_ERR, "lun is out of range (0-%d)", MAX_LUN-1);
return -1; return -1;
} }
if (p->netfn >= 0 && p->lun < 0) { if (p->netfn >= 0 && p->lun < 0) {
printf("if netfn is set, lun must be set also\n"); lprintf(LOG_ERR, "if netfn is set, so must be lun");
return -1; return -1;
} }
if (p->command >= 0 && p->netfn < 0) { if (p->command >= 0 && p->netfn < 0) {
printf("if command is set, netfn must be set also\n"); lprintf(LOG_ERR, "if command is set, so must be netfn");
return -1; return -1;
} }
if (p->subfn >= 0 && p->command < 0) { if (p->subfn >= 0 && p->command < 0) {
printf("if subfn is set, command must be set also\n"); lprintf(LOG_ERR, "if subfn is set, so must be command");
return -1; return -1;
} }
return 0; return 0;