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: 50 - ipmi_hpmfwupg needs a clean up

Browse Source 
HpmfwupgGetBuffersFromFile() - simplify function in order to kill unnecessary
indent. Also, notify user about malloc failure and exit early.
This commit is contained in:
Zdenek Styblik 2013-10-10 10:17:33 +00:00
parent ada4b48baf
commit 29af592773
1 changed files with 23 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
ipmitool/lib/ipmi_hpmfwupg.c
View File

@ -1534,33 +1534,32 @@ HpmfwupgGetBufferFromFile(char *imageFilename,
int ret = 0;
FILE *pImageFile = fopen(imageFilename, "rb");
if (pImageFile == NULL) {
lprintf(LOG_NOTICE, "Cannot open image file %s",
lprintf(LOG_ERR, "Cannot open image file '%s'",
imageFilename);
rc = HPMFWUPG_ERROR;
return HPMFWUPG_ERROR;
}
if (rc == HPMFWUPG_SUCCESS) {
/* Get the raw data in file */
fseek(pImageFile, 0, SEEK_END);
pFwupgCtx->imageSize = ftell(pImageFile);
pFwupgCtx->pImageData = malloc(sizeof(unsigned char)*pFwupgCtx->imageSize);
if (pFwupgCtx->pImageData == NULL) {
lprintf(LOG_ERR, "ipmitool: malloc failure");
fclose(pImageFile);
return HPMFWUPG_ERROR;
}
rewind(pImageFile);
if (pFwupgCtx->pImageData != NULL) {
ret = fread(pFwupgCtx->pImageData,
sizeof(unsigned char),
pFwupgCtx->imageSize,
pImageFile);
if (ret != pFwupgCtx->imageSize) {
lprintf(LOG_ERROR,
lprintf(LOG_ERR,
"Failed to read file %s size %d",
imageFilename,
pFwupgCtx->imageSize);
rc = HPMFWUPG_ERROR;
}
} else {
rc = HPMFWUPG_ERROR;
}
fclose(pImageFile);
}
return rc;
}