i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

CID 1249930: Don't scanf uninitialized buffer

Browse Source 
Would happen in case of a missing config file, for instance.
This commit is contained in:
Philippe Saint-Pierre
2015-06-25 19:29:13 -04:00
parent 6b05e9db3d
commit 1443b2798f
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/preferences/media/MidiSettingsView.cpp
+11 -11
View File
@@ -130,19 +130,19 @@ MidiSettingsView::_LoadSettings()
if (find_directory(B_USER_SETTINGS_DIRECTORY, &path) == B_OK) { if (find_directory(B_USER_SETTINGS_DIRECTORY, &path) == B_OK) {
path.Append(SETTINGS_FILE); path.Append(SETTINGS_FILE);
BFile file(path.Path(), B_READ_ONLY); BFile file(path.Path(), B_READ_ONLY);
if (file.InitCheck() == B_OK) if (file.InitCheck() == B_OK) {
file.Read(buffer, sizeof(buffer)); file.Read(buffer, sizeof(buffer));
} char soundFont[512];
sscanf(buffer, "# Midi Settings\n soundfont = %s\n",
soundFont);
char soundFont[512]; for (int32 i = 0; i < fListView->CountItems(); i++) {
sscanf(buffer, "# Midi Settings\n soundfont = %s\n", BStringItem* item = (BStringItem*)fListView->ItemAt(i);
soundFont); if (!strcmp(item->Text(), soundFont)) {
fListView->Select(i);
for (int32 i = 0; i < fListView->CountItems(); i++) { break;
BStringItem* item = (BStringItem*)fListView->ItemAt(i); }
if (!strcmp(item->Text(), soundFont)) { }
fListView->Select(i);
break;
} }
} }
} }