From 32c46909501aef0a2c870afe919d5d15bac786f1 Mon Sep 17 00:00:00 2001 From: Zdenek Styblik Date: Thu, 22 Jan 2015 12:55:26 +0100 Subject: [PATCH] ID:355 - Fix comparison of unsigned expression Commit fixes comparison of unsigned expression in ipmi_firewall_reset() and ipmi_firewall_enable_disable(). --- lib/ipmi_firewall.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/ipmi_firewall.c b/lib/ipmi_firewall.c index 5ea67ef..c3f51ad 100644 --- a/lib/ipmi_firewall.c +++ b/lib/ipmi_firewall.c @@ -1014,7 +1014,8 @@ ipmi_firewall_enable_disable(struct ipmi_intf * intf, int enable, int argc, char { struct ipmi_function_params p = {0xe, -1, -1, -1, -1}; struct bmc_fn_support * bmc_fn_support; - unsigned int l, n, c, ret; + int ret; + unsigned int l, n, c; unsigned char enables[MAX_COMMAND_BYTES]; if (argc < 1 || strncmp(argv[0], "help", 4) == 0) { @@ -1104,7 +1105,8 @@ ipmi_firewall_reset(struct ipmi_intf * intf, int argc, char ** argv) { struct ipmi_function_params p = {0xe, -1, -1, -1, -1}; struct bmc_fn_support * bmc_fn_support; - unsigned int l, n, c, ret; + int ret; + unsigned int l, n, c; unsigned char enables[MAX_COMMAND_BYTES]; if (argc < 1) {