diff options
author | Ladislav Zezula <ladislav.zezula@avg.com> | 2015-05-28 13:49:23 +0200 |
---|---|---|
committer | Ladislav Zezula <ladislav.zezula@avg.com> | 2015-05-28 13:49:23 +0200 |
commit | 1b38ceb0d4bb4ae32cb93c295e3ef493b91f9a78 (patch) | |
tree | 5634e1d3fd17386975db1c0d4e95176db098bc1f /src/SFileReadFile.cpp | |
parent | c26e12c79f2a5e0c092de4a62565bdae4bf5a7dd (diff) |
+ Fixed defects found by Coverity (well, most of them)
Diffstat (limited to 'src/SFileReadFile.cpp')
-rw-r--r-- | src/SFileReadFile.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/SFileReadFile.cpp b/src/SFileReadFile.cpp index d81b4de..5d64626 100644 --- a/src/SFileReadFile.cpp +++ b/src/SFileReadFile.cpp @@ -54,7 +54,7 @@ static int ReadMpqSectors(TMPQFile * hf, LPBYTE pbBuffer, DWORD dwByteOffset, DW if(hf->SectorOffsets == NULL) { nError = AllocateSectorOffsets(hf, true); - if(nError != ERROR_SUCCESS) + if(nError != ERROR_SUCCESS || hf->SectorOffsets == NULL) return nError; } @@ -231,7 +231,7 @@ static int ReadMpqFileSingleUnit(TMPQFile * hf, void * pvBuffer, DWORD dwFilePos if(hf->pbFileSector == NULL) { nError = AllocateSectorBuffer(hf); - if(nError != ERROR_SUCCESS) + if(nError != ERROR_SUCCESS || hf->pbFileSector == NULL) return nError; pbRawData = hf->pbFileSector; } @@ -367,7 +367,7 @@ static int ReadMpkFileSingleUnit(TMPQFile * hf, void * pvBuffer, DWORD dwFilePos if(hf->pbFileSector == NULL) { nError = AllocateSectorBuffer(hf); - if(nError != ERROR_SUCCESS) + if(nError != ERROR_SUCCESS || hf->pbFileSector == NULL) return nError; pbRawData = hf->pbFileSector; } @@ -477,7 +477,7 @@ static int ReadMpqFileSectorFile(TMPQFile * hf, void * pvBuffer, DWORD dwFilePos if(hf->pbFileSector == NULL) { nError = AllocateSectorBuffer(hf); - if(nError != ERROR_SUCCESS) + if(nError != ERROR_SUCCESS || hf->pbFileSector == NULL) return nError; } @@ -690,7 +690,7 @@ bool WINAPI SFileReadFile(HANDLE hFile, void * pvBuffer, DWORD dwToRead, LPDWORD if(hf->pFileEntry != NULL && (hf->pFileEntry->dwFlags & MPQ_FILE_PATCH_FILE) && hf->pPatchInfo == NULL) { nError = AllocatePatchInfo(hf, true); - if(nError != ERROR_SUCCESS) + if(nError != ERROR_SUCCESS || hf->pPatchInfo == NULL) { SetLastError(nError); return false; |