summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Venture <venture@google.com>2018-12-04 09:52:54 -0800
committerAlexander Amelkin <mocbuhtig@amelkin.msk.ru>2019-02-13 21:28:14 +0300
commit8991f5a9a0c6a4dc13732c940bf92f55e890c9ea (patch)
tree4e49e7eba26bb475dfa846a60bb90ab8f4622b98
parent49384e47beeb8b1c08c2846129369e05bc832c74 (diff)
downloadipmitool-8991f5a9a0c6a4dc13732c940bf92f55e890c9ea.tar.gz
fru: change ipmi_fru_query_new_value to return bool
Change ipmi_fru_query_new_value to return the bool type instead of an int that's being used as a boolean value. Signed-off-by: Patrick Venture <venture@google.com>
-rw-r--r--lib/ipmi_fru.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/ipmi_fru.c b/lib/ipmi_fru.c
index 5e3fde3..9f59283 100644
--- a/lib/ipmi_fru.c
+++ b/lib/ipmi_fru.c
@@ -1487,17 +1487,17 @@ fru_area_print_multirec(struct ipmi_intf * intf, struct fru_info * fru,
* returns : FALSE if data not changed
*/
static
-int
+bool
ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len)
{
- int status=FALSE;
+ bool status = false;
int ret;
char answer;
printf("Would you like to change this value <y/n> ? ");
ret = scanf("%c", &answer);
if (ret != 1) {
- return FALSE;
+ return false;
}
if( answer == 'y' || answer == 'Y' ){
@@ -1514,7 +1514,7 @@ ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len)
ret = scanf("%x", holder+i);
if (ret != 1) {
free(holder);
- return FALSE;
+ return false;
}
}
for( i=0;i<len;i++ ){
@@ -1523,7 +1523,7 @@ ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len)
/* &data[offset++] */
free(holder);
holder = NULL;
- status = TRUE;
+ status = true;
}
else{
printf("Entered %c\n",answer);