Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] coverity issues #12375

Merged
merged 9 commits into from Jun 30, 2017

[fix] CID 719031: Copy into fixed size buffer

  • Loading branch information...
Rechi committed Jun 30, 2017
commit e89e98dcaa2d5b86714c96c0b35c2a72e53c8fe5
@@ -647,7 +647,8 @@ HANDLE iso9660::FindFirstFile( char *szLocalFolder, WIN32_FIND_DATA *wfdFile )
#ifdef TARGET_WINDOWS
wcscpy_s(wfdFile->cFileName, MAX_PATH, KODI::PLATFORM::WINDOWS::ToW(m_searchpointer->name).c_str());
#else
strcpy(wfdFile->cFileName, m_searchpointer->name );
strncpy(wfdFile->cFileName, m_searchpointer->name, sizeof(wfdFile->cFileName) - 1);
wfdFile->cFileName[sizeof(wfdFile->cFileName) - 1] = '\0';
#endif

if ( m_searchpointer->type == 2 )
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.