empathicqubit/AppleWin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[PVS-Studio] Fix assignment in DenibblizeTrack() and potential buffer overflow in SetCurrentImageDir()

Browse source
This commit is contained in:
michaelangel007 2015-07-30 07:28:20 -07:00
parent 5f1decaaea
commit 2f645d68f4
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
source/Applewin.cpp
View file

@ -550,7 +550,7 @@ void SetCurrentImageDir(const char* pszImageDir)
strcpy(g_sCurrentDir, pszImageDir); strcpy(g_sCurrentDir, pszImageDir);
int nLen = strlen( g_sCurrentDir ); int nLen = strlen( g_sCurrentDir );
if( g_sCurrentDir[ nLen - 1 ] != '\\' ) if ((nLen > 0) && (g_sCurrentDir[ nLen - 1 ] != '\\'))
{ {
g_sCurrentDir[ nLen + 0 ] = '\\'; g_sCurrentDir[ nLen + 0 ] = '\\';
g_sCurrentDir[ nLen + 1 ] = 0; g_sCurrentDir[ nLen + 1 ] = 0;

2
source/DiskImageHelper.cpp
View file

@ -436,7 +436,7 @@ void CImageBase::DenibblizeTrack(LPBYTE trackimage, SectorOrder_e SectorOrder, i
offset = 0; offset = 0;
} }
if ((bytenum == 3) && (byteval[1] = 0xAA)) if ((bytenum == 3) && (byteval[1] == 0xAA))
{ {
int loop = 0; int loop = 0;
int tempoffset = offset; int tempoffset = offset;